Skip to content

Conversation

@NARUDESIGNS
Copy link
Collaborator

Remade the wysiwyg jest test conversion due to issues from former PR to see if all tests work fine.
Finger crossed 🤞

@gitpod-io
Copy link

gitpod-io bot commented Jan 19, 2022

@jywarren
Copy link
Member

This is really so strange Paul I'm sorry. Somehow this test is affecting other files and I think we have to figure out why or how that could happen. What if we actually remove all of the content of the tests or even just all of the tests themselves or we test something obvious like true is true. That way I mean if it's a completely obvious test it doesn't seem like there's any way I could affect other test files right? Do you want to give that a try in this pull request?

@NARUDESIGNS
Copy link
Collaborator Author

Sure, it's something I can try. Although, I don't know why it's particular to this test. You can see that this PR - #799 was made today and all test passes. But we could try your idea nonetheless.

@NARUDESIGNS
Copy link
Collaborator Author

@jywarren I just tried pushing the files with

expect(true).toBe(true);

But it still fails. I have another idea. I going to create a new file entirely in a new branch, add this test case and then push to see if it passes. If it does, I'd go on to add a test case and push changes to see again if it passes.
My only concern is that it would mean I've made 3 PRs just trying to debug this issue.

@NARUDESIGNS
Copy link
Collaborator Author

solved @ #785

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.

3 participants