Integration test

[junip]

Integration Tests for the following

• User can click on mentors channel.
• Verification of country/technology filter

resolve #633

[junip]

@brentmclark can you please review the changes. Thanks

[brentmclark]

@junipdewan I like the code, but some of these tests are failing. Have you been able to figure out why yet? I suspect there are some environment-specific differences between local and staging.

[junip]

@brentmclark , I have tried to figure out the cause but didn't find any. I have tried by changing env file and pointing API ENDPOINT from staging to production it's working fine.

[brentmclark]

I'll have a look at my earliest opportunity.

[brentmclark]

Quick update; I'm not having a ton of luck hunting this down. My best guess at this point is some form of data variance in the CI environment and the local environment. A decent path forward, in that case, would be to use a fixture to control the API response. This would eliminate any variance between environments.

More updates when I can, unless you want to take a crack at adding a fixture @junipdewan?

[moshfeu]

@junipdewan what do you think?

[junip]

@moshfeu @brentmclark , Sure I will add the fixture to control the API response and update the PR accordingly.

[brentmclark]

@junipdewan I added fixtures for the /mentors endpoint in #656; you may want to wait for that to be merged before moving forward here. It includes a fix for the huge gotcha that cypress does not have native support for window.fetch 🙃

[junip]

@brentmclark Thanks for your update. As I can see #656 is merged. I will work on this current PR and update you.

[junip]

@brentmclark I have updated the test cases for mentors spec. Can you please review the changes ?

[brentmclark]

This is so great; thank you @junipdewan! If you don't mind touching up those 3 spots I called out that would be most appreciated. I'm ready to approve after that!

[junip]

@brentmclark, I have added the changes as per your review. Also added test cases

• Like / Unlike the Mentor

[brentmclark]

@junipdewan Great work on the changes and the new tests! One of the new tests is failing in CI land, unfortunately. Does anything fail locally?

[junip]

@brentmclark No. All test were working fine.So, I have rechecked the test cases again locally but all test cases are passed.

[junip]

@brentmclark fixed the failed test cases.

[brentmclark]

Looks good to me! @moshfeu does everything look good on your end?

[junip]

@brentmclark , @moshfeu. JFYI there is already an opened like/unlike issue. If everything is fine in this PR we can close the issue. Thanks

[moshfeu]

Looking good to me as well :)
This was a long trip.
Thank you and, of course, @brentmclark ❤️

[brentmclark]

Great work and impressive dedication @junipdewan. Thanks a ton for your patience as I worked through the core issues with auth in Cypress. I'm so excited to have these tests in place; they will be crucial as the app is enhanced.

🎉