search

tripal / t4d8

This is a temporary repository for Drupal 9 upgrade for Tripal v4. It is meant to house issues related to the upgrade.
GNU General Public License v2.0
1 stars 10 forks source link

Updates to Entities, Bundles, and Fields #1

Closed shawnawsu closed 1 year ago

shawnawsu commented 5 years ago

Please tag all tickets and pr's with group 1 to denote it relates to this first group of updates.

Files this update will require to be reviewed and updated (in order):

laceysanderson commented 5 years ago

PR: #10

Files this PR upgrades

Method

Tests

TripalVocabEntityTest.php:

testTripalVocabEntity()

  1. Go to Vocab listing as unprivileged user and ensure they get access denied.
  2. Login as privileged user; check that "Tripal Controlled Vocabularies" is in the structure listing.
  3. Click the link and check it goes to the vocab listing. Also ensure users get a message telling them there is no content yet.
  4. Goto the add vocabulary form. Check it has the form fields we expect. Fill it out with a valid unique name and submit. Check for the message indicating the vocab is saved.
  5. Go to the Vocab listing. Check there is no message indicating no vocabularies. Also check the name, edit and delete links for the new vocabulary are present in the page.
  6. Click the edit link, change the vocabulary name and submit. Check for the message indicating the vocab changes were saved.
  7. Go to the Vocab listing. Check the new name, edit and delete links for the changed vocabulary are present in the page. Also check that the old name is not present in the page.
  8. Click the delete link and check you get the confirmation page. Leave the delete page without submitting and ensure the vocab wasn't deleted on the listing page.
  9. Click delete again. This time confirm delete and check the listing to ensure the vocab is no longer there.

testPaths()

For Each route defined for TripalVocab crud ensure a user logged in with a specific set of permissions gets the page status code we expect (i.e. access denied if they don't have sufficient permission and 200 if they do). See providerTestPaths() for the list of paths tested.

TripalTermEntityTest.php:

testTripalTermEntity()

  1. Go to Term listing as unprivileged user and ensure they get access denied.
  2. Login as privileged user; check that "Tripal Controlled Vocabulary Terms" is in the structure listing.
  3. Click the link and check it goes to the term listing. Also ensure users get a message telling them there is no content yet.
  4. Goto the add vocabulary term form. Check it has the form fields we expect. Fill it out with a valid unique name, accession and newly created vocab and submit. Check for the message indicating the vocab term is saved.
  5. Go to the Term listing. Check there is no message indicating no terms. Also check the name, accession, vocab, edit and delete links for the new term are present in the page.
  6. Click the edit link, change the term name and submit. Check for the message indicating the term changes were saved.
  7. Go to the term listing. Check the new name, old accession, old vocab, edit and delete links for the changed vocabulary are present in the page.
  8. Click the delete link and check you get the confirmation page. Leave the delete page without submitting and ensure the term wasn't deleted on the listing page.
  9. Click delete again. This time confirm delete and check the listing to ensure the vocab is no longer there.

testPaths()

For Each route defined for TripalTerm crud ensure a user logged in with a specific set of permissions gets the page status code we expect (i.e. access denied if they don't have sufficient permission and 200 if they do). See providerTestPaths() for the list of paths tested.

laceysanderson commented 4 years ago

PR #13 PR #14 PR #15

Tripal Entity Type and Tripal Entity

Tripal content and tripal content types are now fully functional! Additionally, I expanded Tripal Vocab and Tripal Term to include all details needed independent of Chado and added view pages. I l also tested basic views integration and it works!

Files these PRs upgrade

laceysanderson commented 4 years ago

I've gone through and created issues for all the remaining parts of group 1. You can access them using the group 1 tag or by checking the main list at the top of this page which I updated with the sub-issues.

To jump in, mention your intent on one of the sub-issues and once complete, report back here :-) Also if you have any questions or run into any problems feel free to tag me!

laceysanderson commented 1 year ago

Closing these roadmap issues as we arn't currently using this approach to focus the upgrade.

This is part of an attempt to minimize the number of issues in this repo in preparation for moving back to the core tripal/tripal repo.