Skip to content
Snippets Groups Projects

Remove extra action test assertions

Merged Tamlyn Rhodes requested to merge 52-action-tests into master

These tested previous client-server interactions but in a way that failures were not reported.

Making them pass again would mean mocking the return data from the server but since this is what these assertions were testing, it would not have been useful.

If these tests worked with real server interaction, they would have been useful. They should be replaced with end-to-end tests, probably in pubsweet-starter.

Closes #52

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
Please register or sign in to reply