Smoke testing the CoLab instance on the cloud68 servers
Description; further requests from the client after testing.
Please see updated fields in NCRC_updated sheet, highlighted in yellow.
See test feedback document; Testing.xlsx
Acceptance criteria (feedback verbatim from the client);
Beatrice-Emanuela Suarasan (5aac5fbe) at 30 Jul 10:53
test: update form tests & fix formatting
Currently, the tests are using localhost
as their target url - the prod environment is build & then used for testing inside the CI, much like the local environment has to be up & running before running the tests.
In order for the tests to be run directly on a given url (e.g. https://colab.cloud68.co/), there should be cypress config files created for each instance, which should contain the urls needed to run the tests.
Given the fact that the CI tests also directly use the DB in order to populate tables with test data, this would also mean editing the tests to run in a way which does not affect the actual prod users' data (e.g. user accounts, user manuscripts, etc.) or removing the direct DB commands entirely & run the tests much like a normal user would (i.e. without having DB access)
Severity: High
Environment: Prod
Steps to reproduce:
Expected results: The width should be set to 25%
Actual results: The current width is set to 14%
@ryandixpeek should this change be available on instances other than Aperture?
As reviewer I would like to see Evaluation summery in the review page so that I can reed the final decision of the team work
Acceptance criteria:
Severity: High
Environment: Prod
Steps to reproduce:
Expected results: The width should be set to 25%
Actual results: The current width is set to 14%
Smoke testing the NCRC instance on the cloud68 servers
@kominoshja yep, publishing works now. I'll just log a bug for the first column issue, so we can close this ticket. is that alright with you too, @ryandixpeek ?
Beatrice-Emanuela Suarasan (32484555) at 29 Jul 11:08
test: add decision block tests & update tests
As Editor, I would like to see notification logs, in order to capture a record of communication that was sent within the context of each article.
Acceptance criteria:
Beatrice-Emanuela Suarasan (7e997066) at 29 Jul 09:31
test: add notification logs tests & update other colab tests
Severity: Medium
Environment: Local & Prod
Description: Currently, on prod, one of the imported articles is missing its title ( https://ncrc.kotahi.cloud/kotahi/versions/3c944e83-ef76-4053-ba98-09cf2a52f0bc/evaluation ) On closer inspection, it would seem this is the only article containing HTML tags in the title (compared to the other imported articles), as per the PubMed API of this specific article ( https://eutils.ncbi.nlm.nih.gov/entrez/eutils/esummary.fcgi?db=pubmed&id=34253645&retmode=json&tool=my_tool&email=my_email@example.com )
Desired outcome: We should be able to ignore the HTML tags (if any) & import the title
Beatrice-Emanuela Suarasan (6a3f6cda) at 28 Jul 11:33
test: update colab control page tests
Beatrice-Emanuela Suarasan (abc7e7a8) at 28 Jul 08:59
test: update ncrc 05 file
@kominoshja to recap, the following is still not working:
Based on what Mihail & I have discussed, there seems to be a misconfiguration of the .env vars which causes both to happen.
@kominoshja please keep in mind that the latest changes also have to be pushed - currently, the publishing feature is not working (we implemented the publishing feature a few days ago, when we changed the spreadsheet endpoint - #559 (closed)), so we need the latest changes for that to work
@kominoshja I've just checked NCRC - which uses the MANUSCRIPTS_TABLE_FIRST_COLUMN_WIDTH=25% env var - and the value is still 14%