Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy to Production #2740

Merged
merged 3 commits into from
Apr 24, 2024
Merged

Deploy to Production #2740

merged 3 commits into from
Apr 24, 2024

Conversation

cesarvarela
Copy link
Collaborator

@cesarvarela cesarvarela commented Apr 22, 2024

* Forward graphql requests to netlify function

* Trigger Build

* update lockfile

* Avoid having to introspect schema for every request

* Allow anonymous read only queries

* Allow anonymous read only queries

* Trigger Build
Copy link

codecov bot commented Apr 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.30%. Comparing base (e834df1) to head (94f83e2).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2740      +/-   ##
==========================================
+ Coverage   86.21%   86.30%   +0.08%     
==========================================
  Files         248      248              
  Lines        8338     8338              
  Branches     2626     2628       +2     
==========================================
+ Hits         7189     7196       +7     
+ Misses       1055     1049       -6     
+ Partials       94       93       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

cypress bot commented Apr 22, 2024

43 failed tests on run #2313 ↗︎

43 438 244 0 Flakiness 0

Details:

Merge pull request #2740 from responsible-ai-collaborative/staging
Project: aiid-site Commit: e6310ecc45
Status: Failed Duration: 22:36 💡
Started: May 4, 2024 6:19 AM Ended: May 4, 2024 6:42 AM
Failed  cite.cy.js • 2 failed tests • Cypress e2e tests

View Output

Test Artifacts
Cite pages > Should remove duplicate Test Replay
Cite pages > Should clone incident opening the preloaded new Incident form Test Replay
Failed  pages.cy.js • 27 failed tests • Cypress e2e tests

View Output

Test Artifacts
Pages > /ja/ Should have open graph tags Test Replay
Pages > /ja/taxonomies/ Should have open graph tags Test Replay
Pages > /ja/apps/discover/ Should have open graph tags Test Replay
Pages > /ja/apps/incidents/ Should have open graph tags Test Replay
Pages > /ja/apps/newsdigest/ Should have open graph tags Test Replay
Pages > /ja/apps/submit/ Should have open graph tags Test Replay
Pages > /ja/apps/submitted/ Should have open graph tags Test Replay
Pages > /ja/apps/variants/ Should have open graph tags Test Replay
Pages > /ja/blog/ Should have open graph tags Test Replay
Pages > /ja/summaries/flagged/ Should have open graph tags Test Replay
The first 10 failed tests are shown, see all 27 tests in Cypress Cloud.
Failed  submit.cy.js • 2 failed tests • Cypress e2e tests

View Output

Test Artifacts
The Submit form > Should *not* show related orphan reports Test Replay
The Submit form > Should hide incident_date, description, deployers, developers & harmed_parties if incident_ids is set Test Replay
Failed  integrity.cy.js • 4 failed tests • Cypress e2e tests

View Output

Test Artifacts
Integrity > Shouldn't have repeated report numbers Test Replay
Integrity > is_incident_report should be true for reports assigned to incidents and vice versa Test Replay
Integrity > Classifications should be linked to one and only one incident Test Replay
Integrity > Classifications should only contain attributes defined in the taxonomy Test Replay
Failed  incidents/history.cy.js • 2 failed tests • Cypress e2e tests

View Output

Test Artifacts
Incidents > Should display an error message if no Incident ID is provided Test Replay
Incidents > Should display an error message if an invalid Incident ID is provided Test Replay

The first 5 failed specs are shown, see all 9 specs in Cypress Cloud.

Review all test suite changes for PR #2740 ↗︎

Copy link

@cesarvarela
Copy link
Collaborator Author

Lets wait for this one to work before pushing.

Copy link

@cesarvarela cesarvarela marked this pull request as ready for review April 23, 2024 21:29
@cesarvarela
Copy link
Collaborator Author

@kepae did some smoke testing and everything seems to be working

@kepae
Copy link
Collaborator

kepae commented Apr 23, 2024

@cesarvarela Happy to push tonight, but perhaps tomorrow is better when I'm more available. Just let me know.

@cesarvarela
Copy link
Collaborator Author

yes tomorrow sounds better

@kepae
Copy link
Collaborator

kepae commented Apr 24, 2024

@cesarvarela Pushing!

@kepae kepae merged commit e6310ec into master Apr 24, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants