- BugFix: allow tested_reference_str to be None

[SSE4]

closes: #10714

Changelog: Feature: Allow tested_reference_str to be None.
Docs: omit

• Refer to the issue that supports this Pull Request.
• If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
• I've read the Contributing guide.
• I've followed the PEP8 style guides for Python code.
• I've opened another PR in the Conan docs repo to the develop branch, documenting this one.

^{Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.}

[lasote]

We would like to avoid that new argument. We've been discussing and we might prefer if self.requires(None) and self.build_requires(None) don't do anything, that is actually more accurate to the current behavior with 1.X, because the whole graph is not computed either, just the test_package and its requires. Probably we need to initialize the self.tested_reference_str to None always. WDYT?

[SSE4]

We would like to avoid that new argument. We've been discussing and we might prefer if self.requires(None) and self.build_requires(None) don't do anything, that is actually more accurate to the current behavior with 1.X, because the whole graph is not computed either, just the test_package and its requires. Probably we need to initialize the self.tested_reference_str to None always. WDYT?

I am personally fine with that. it will allow to compute requirements of test_package in our CI, and we already have requirements of the package itself computed. (well, it might not work if test_package changes options of requirements, but it doesn't matter for CCI in this case, as it will fail for a different reason - missing package, cause package built is not one being tested).

[SSE4]

@lasote I've changed the implementation according to your suggestion. tested_reference_str is initialized to None. self.requires(None) and self.build_requires(None) are no-ops now.

[memsharded]

Not worth the trick. Just write a explicit conanfile it will be just a couple of lines longer, but way more explicit and understandable.

[SSE4]

okay, changed