Fix python unit test

[mbertrand]

What are the relevant tickets?

N/A

Description (What does it do?)

Fixes a flaky test temporarily (long term fix: get rid of n+1 queries for this view)

How can this be tested?

Tests should all pass

[ChristopherChudzicki]

This improves the situation, so 👍

I did notice two things:

First, lots of

  /src/learning_resources/views_test.py:1245: PytestUnknownMarkWarning: Unknown pytest.mark.skip_nplusone_check - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @pytest.mark.skip_nplusone_check

Second: test_similar_resources_endpoint_does_not_return_self is similarly flakey. Not sure if it's the same issue, though.

[mbertrand]

https://github.com/mitodl/mit-learn/pull/2605/files#diff-4ff824f0186901e292c8550b4150c3c05f94653f487c498a3dc384a4ff998da5

Temporary decorator to skip n+1 checks

[ChristopherChudzicki]

@mbertrand I merged

• Fixes flaky test on resource cards #2628

which fixes that failing JS test.

It looks like in mitxonline the skip_nplusone_check is registered in pytest.ini but not in Learn. Maybe we should add it to reduce the warnings?

[mbertrand]

It looks like in mitxonline the skip_nplusone_check is registered in pytest.ini but not in Learn. Maybe we should add it to reduce the warnings?

Added