Fix potential None and PackageType comparison when deducing cpp_info (#19494)

* Fix None comparison with potential PackageType in components

* Invert order

* simplify

* rename test

* Fix

Author: AbrilRBS

conan/internal/model/cpp_info.py

@@ -659,7 +659,7 @@ def deduce_locations(self, conanfile, component_name=""):
             return
 
         # automatic location deduction from a single .lib=["lib"]
-        if self._type not in [None, PackageType.SHARED, PackageType.STATIC]:
+        if self._type is not None and self._type not in [PackageType.SHARED, PackageType.STATIC]:
             raise ConanException(f"{name} has a library but .type {self._type} is not static/shared")
 
         # If no location is defined, it's time to guess the location

test/integration/toolchains/cmake/cmakedeps2/test_cmakedeps.py

@@ -391,6 +391,17 @@ def package_info(self):
            "             $<$<CONFIG:RELEASE>:MyOpenMPILib>)" in dep
 
 
+def test_cmake_component_type_none_check():
+    tc = TestClient()
+    dep = (GenConanfile("dep", "0.1")
+           .with_package_file("lib/libmain.so", "dynamic library")
+           .with_package_info({"components": {"main": {"libs": ["libmain.so"], "type": "'shared-library'"}}}))
+    tc.save({"conanfile.py": dep})
+    tc.run("create")
+    tc.run("install --requires=dep/0.1 -g CMakeConfigDeps")
+    assert "None is not a valid PackageType" not in tc.out
+
+
 def test_cmake_extra_dependencies_components():
     tc = TestClient()
     dep = textwrap.dedent("""