#4655 Add link to node/123 to dataset import status table.

[swirtSJW]

Fixes #4655

Describe your changes

QA Steps

• git checkout 4655-add-node-link-to-import-status
• drush cr to pick up twig changes
• visit /dashboard/datastore
• validate that the dataset identifier comes after/below the title
• validate that the dataset title has no link
• validate the "view" links to the FE version of the dataset page
• validate the "admin" links to the node version of the dataset page
• inspect the source and validate that the "admin" link has an aria-label that reads "admin: "
• inspect the source and validate that the "view" link has an aria-label that reads "view: "

Checklist before requesting review

If any of these are left unchecked, please provide an explanation

• I have updated or added tests to cover my code
• I have updated or added documentation

[dafeder]

Code itself is fine, but I'm not sure why you added the node_id to the data in all the tests but then never have assertions for it?

[swirtSJW]

Code itself is fine, but I'm not sure why you added the node_id to the data in all the tests but then never have assertions for it?

I had to add the node_id to the tests because otherwise the template errors out if the node_id does not exist in the clall to path().

I can add additional assertions to make sure that the twig path() works, but that seems excessive to test that a core twig function does its job. That is all it would test since it blows up if you feed it something that is not a node id.

[dafeder]

Approved, but holding off on merging until 4.x