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
feat: shard modules and API integration tests #6775
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
527f001
to
05703d3
Compare
05703d3
to
e7759d1
Compare
d84a99f
to
a3ae5d4
Compare
4db2232
to
4d6e7d8
Compare
4d6e7d8
to
efc6145
Compare
@adrien2p thanks that makes sense, I’ll try migrating the flaky ones to see if that fixes it. |
@adrien2p, one of the skipped tests (cart.js) is using the |
Yes ill have a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing stuff, such an improvement 👍
@@ -65,7 +65,7 @@ medusaIntegrationTestRunner({ | |||
}) | |||
|
|||
describe("POST /store/carts", () => { | |||
it("should create a cart", async () => { | |||
it.skip("should create a cart", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's just follow-up and re-enable these, but it shouldn't be a blocker IMO
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it is not for that pr, I am investigating it but if there is an issue it is separate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, would also just get this in now, and resolve (future) flaky tests in follow-up PRs
@riqwan I am trying to investigate the cart just so you know |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏆
Wasn't expecting an approval yet. 😄 I had some more things to do here, but it might be a good idea to take this out for a spin. We will still be limited by the unit tests, it takes about 20 mins. I wanted to split that into 3 buckets as well, but I need to brush up on my bash skills and configure turbo to filter them. But that can come as a follow up! 🤞🏻 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing 😍
what: