Skip to content

Conversation

nickserv
Copy link
Member

@nickserv nickserv commented Nov 7, 2019

#53

  • Test basic button injection
  • Create fully-functioning stubs
    • chrome.extension.storage.sync
    • MutationObserver
  • Attempt to test with mocked window.location
  • Test option loading behavior
  • Consider adding more granular unit tests per class/file/method/function
  • Consider using dom-testing-library for better assertions and consistency with Testing options #87
  • Complete test coverage

@kaicataldo
Copy link
Member

Can we fix the merge conflicts and merge this? If everything is set up, I don't think we need to wait until we have complete coverage before merging.

@kaicataldo

This comment has been minimized.

@nickserv nickserv marked this pull request as ready for review August 29, 2020 23:09
@nickserv
Copy link
Member Author

nickserv commented Aug 30, 2020

I'm going to see if I can swap in sinon-chrome similarly to #87 before I merge this. Maintaining mocks manually is a lot of churn.

Base automatically changed from master to main February 1, 2021 03:57
@kaicataldo
Copy link
Member

Are you still working on this? If not, I think it would be best to close the PR for now so others can feel free to work on it (would love to land this PR if you're still working on it, though!).

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