pytest: `'store_true'` does not work correctly in parser.addoption

One more from https://github.com/conda-forge/staged-recipes/pull/16888, where (after patching in workarounds for #9300 & #9301), the test suite failed with

==================================== ERRORS ====================================
________________ ERROR at setup of test_serialization[cpu-gelu] ________________

self = <_pytest.config.Config object at 0x7fc1975d41c0>
name = 'skip_custom_ops', default = <NOTSET>, skip = False

    def getoption(self, name: str, default=notset, skip: bool = False):
        """Return command line option value.
    
        :param name: Name of the option.  You may also specify
            the literal ``--OPT`` option instead of the "dest" option name.
        :param default: Default value if no option of that name exists.
        :param skip: If True, raise pytest.skip if option does not exists
            or has a None value.
        """
        name = self._opt2dest.get(name, name)
        try:
>           val = getattr(self.option, name)
E           AttributeError: 'Namespace' object has no attribute 'skip_custom_ops'

../[...]/_pytest/config/__init__.py:1463: AttributeError

The above exception was the direct cause of the following exception:

request = <SubRequest 'set_global_variables' for <Function test_serialization[cpu-gelu]>>

    @pytest.fixture(scope="session", autouse=True)
    def set_global_variables(request):
>       if request.config.getoption("--skip-custom-ops"):
E       ValueError: no option named 'skip_custom_ops

So from that, it looks like something is going wrong with skip_custom_ops. Chasing around the upstream repo I find that this option is added here as follows:

def pytest_addoption(parser):
    parser.addoption(
        "--skip-custom-ops",
        action="store_true",
        help="When a custom op is being loaded in a test, skip this test.",
    )

The pytest docs for parser.addoption say that this works like argparse, where the linked docs say:

'store_true' and 'store_false' - These are special cases of 'store_const' used for storing the values True and False respectively. In addition, they create default values of False and True respectively.

What caught my eye in the stack trace was the default = <NOTSET>, which is clearly at odds with that documentation. To verify this hypothesis, I added the following patch

diff --git a/tensorflow_addons/utils/test_utils.py b/tensorflow_addons/utils/test_utils.py
index 9d1dec0..0efd707 100644
--- a/tensorflow_addons/utils/test_utils.py
+++ b/tensorflow_addons/utils/test_utils.py
@@ -134,7 +134,8 @@ def set_seeds():
 def pytest_addoption(parser):
     parser.addoption(
         "--skip-custom-ops",
-        action="store_true",
+        action="store",
+        default=True,
         help="When a custom op is being loaded in a test, skip this test.",
     )

and voilà, it works.