Ensure CMakeLists don't miss any files

samples/integration/cmake-fetch-content/CMakeLists.txt

@@ -23,7 +23,7 @@ endif()
 
 add_executable (
      application
-     src/main
+     src/main.cpp
 )
 
 # Link to Azure SDK

samples/integration/vcpkg-all-smoke/CMakeLists.txt

@@ -22,7 +22,7 @@ endif()
 
 add_executable (
   vcpkg-all-smoke
-    src/main
+    src/main.cpp
 )
 
 # Link to Azure SDK

sdk/attestation/azure-security-attestation/CMakeLists.txt

@@ -31,37 +31,37 @@ find_package(OpenSSL REQUIRED)
 
 set(
   AZURE_ATTESTATION_HEADER
+    inc/azure/attestation.hpp
     inc/azure/attestation/attestation_administration_client.hpp
-    inc/azure/attestation/attestation_client.hpp
     inc/azure/attestation/attestation_client_models.hpp
     inc/azure/attestation/attestation_client_options.hpp
+    inc/azure/attestation/attestation_client.hpp
     inc/azure/attestation/dll_import_export.hpp
     inc/azure/attestation/rtti.hpp
-    inc/azure/attestation.hpp
 )
 
 set(
   AZURE_ATTESTATION_SOURCE
-    src/private/crypto/inc/crypto.hpp
-    src/private/crypto/openssl/openssl_helpers.hpp
-    src/private/crypto/openssl/opensslcert.cpp
-    src/private/crypto/openssl/opensslcert.hpp
-    src/private/crypto/openssl/opensslcrypto.cpp
-    src/private/crypto/openssl/opensslkeys.cpp
-    src/private/crypto/openssl/opensslkeys.hpp
+    src/attestation_administration_client.cpp
+    src/attestation_client_options.cpp
+    src/attestation_client.cpp
     src/private/attestation_client_models_private.hpp
     src/private/attestation_client_private.cpp
     src/private/attestation_client_private.hpp
     src/private/attestation_common_request.cpp
     src/private/attestation_common_request.hpp
     src/private/attestation_deserializers_private.cpp
     src/private/attestation_deserializers_private.hpp
-    src/private/jsonhelpers.cpp
     src/private/jsonhelpers_private.hpp
+    src/private/jsonhelpers.cpp
     src/private/package_version.hpp
-    src/attestation_administration_client.cpp
-    src/attestation_client.cpp
-    src/attestation_client_options.cpp
+    src/private/crypto/inc/crypto.hpp
+    src/private/crypto/openssl/openssl_helpers.hpp
+    src/private/crypto/openssl/opensslcert.cpp
+    src/private/crypto/openssl/opensslcert.hpp
+    src/private/crypto/openssl/opensslcrypto.cpp
+    src/private/crypto/openssl/opensslkeys.cpp
+    src/private/crypto/openssl/opensslkeys.hpp
 )
 
 add_library(azure-security-attestation ${AZURE_ATTESTATION_HEADER} ${AZURE_ATTESTATION_SOURCE})

sdk/attestation/azure-security-attestation/samples/policy-certificates/CMakeLists.txt

@@ -10,7 +10,9 @@ set(CMAKE_CXX_STANDARD_REQUIRED True)
 macro (define_sample samplename)
 add_executable (
   attestation-${samplename}
-  ${samplename}.cpp)
+  ${samplename}.cpp
+  cryptohelpers.hpp
+)
 
 CREATE_PER_SERVICE_TARGET_BUILD_FOR_SAMPLE(attestation-${samplename} attestation)
 

sdk/attestation/azure-security-attestation/samples/policy/CMakeLists.txt

@@ -10,7 +10,9 @@ set(CMAKE_CXX_STANDARD_REQUIRED True)
 macro (define_sample samplename)
 add_executable (
   attestation-${samplename}
-  ${samplename}.cpp)
+  ${samplename}.cpp
+  cryptohelpers.hpp
+)
 
 CREATE_PER_SERVICE_TARGET_BUILD_FOR_SAMPLE(attestation attestation-${samplename})
 

sdk/attestation/azure-security-attestation/test/ut/CMakeLists.txt

@@ -24,16 +24,17 @@ include(GoogleTest)
 
 add_executable (
      azure-security-attestation-test
-     macro_guard.cpp
-     crypto_test.cpp
-     attestation_test.cpp
-     token_test.cpp 
-     policygetset_test.cpp
-     attestation_collateral.cpp 
+     attestation_collateral.cpp
      attestation_metadata.cpp
-     policycertmgmt_test.cpp
-     tpmattestation_test.cpp 
+     attestation_test.cpp
      crypto_test_collateral.cpp
+     crypto_test_collateral.hpp
+     crypto_test.cpp
+     macro_guard.cpp
+     policycertmgmt_test.cpp
+     policygetset_test.cpp
+     token_test.cpp
+     tpmattestation_test.cpp
 )
 
 create_per_service_target_build(attestation azure-security-attestation-test)

sdk/core/azure-core-amqp/CMakeLists.txt

@@ -56,39 +56,57 @@ find_package(azure_c_shared_utility CONFIG REQUIRED)
 
 
 set (AZURE_CORE_AMQP_HEADER
+    inc/azure/core/amqp.hpp
     inc/azure/core/amqp/cancellable.hpp
     inc/azure/core/amqp/claims_based_security.hpp
-    inc/azure/core/amqp/common/async_operation_queue.hpp
-    inc/azure/core/amqp/common/completion_operation.hpp
-    inc/azure/core/amqp/common/global_state.hpp
-    inc/azure/core/amqp/connection.hpp
     inc/azure/core/amqp/connection_string_credential.hpp
+    inc/azure/core/amqp/connection.hpp
+    inc/azure/core/amqp/dll_import_export.hpp
+    inc/azure/core/amqp/doxygen_pragma.hpp
     inc/azure/core/amqp/endpoint.hpp
     inc/azure/core/amqp/link.hpp
+    inc/azure/core/amqp/management.hpp
     inc/azure/core/amqp/message_receiver.hpp
     inc/azure/core/amqp/message_sender.hpp
+    inc/azure/core/amqp/rtti.hpp
+    inc/azure/core/amqp/session.hpp
+    inc/azure/core/amqp/common/async_operation_queue.hpp
+    inc/azure/core/amqp/common/completion_operation.hpp
+    inc/azure/core/amqp/common/global_state.hpp
     inc/azure/core/amqp/models/amqp_error.hpp
     inc/azure/core/amqp/models/amqp_header.hpp
     inc/azure/core/amqp/models/amqp_message.hpp
     inc/azure/core/amqp/models/amqp_properties.hpp
-    inc/azure/core/amqp/models/amqp_value.hpp
     inc/azure/core/amqp/models/amqp_protocol.hpp
+    inc/azure/core/amqp/models/amqp_value.hpp
     inc/azure/core/amqp/models/message_source.hpp
     inc/azure/core/amqp/models/message_target.hpp
     inc/azure/core/amqp/models/messaging_values.hpp
-    inc/azure/core/amqp/session.hpp
+    inc/azure/core/amqp/network/amqp_header_detect_transport.hpp
+    inc/azure/core/amqp/network/sasl_transport.hpp
+    inc/azure/core/amqp/network/socket_listener.hpp
+    inc/azure/core/amqp/network/socket_transport.hpp
+    inc/azure/core/amqp/network/tls_transport.hpp
+    inc/azure/core/amqp/network/transport.hpp
 )
 
 set(AZURE_CORE_AMQP_SOURCE
     src/amqp/cancellable.cpp
     src/amqp/claim_based_security.cpp
-    src/amqp/connection.cpp
     src/amqp/connection_string_credential.cpp
+    src/amqp/connection.cpp
     src/amqp/link.cpp
     src/amqp/management.cpp
     src/amqp/message_receiver.cpp
     src/amqp/message_sender.cpp
     src/amqp/session.cpp
+    src/amqp/private/claims_based_security_impl.hpp
+    src/amqp/private/connection_impl.hpp
+    src/amqp/private/link_impl.hpp
+    src/amqp/private/management_impl.hpp
+    src/amqp/private/message_receiver_impl.hpp
+    src/amqp/private/message_sender_impl.hpp
+    src/amqp/private/session_impl.hpp
     src/common/global_state.cpp
     src/models/amqp_error.cpp
     src/models/amqp_header.cpp
@@ -103,7 +121,9 @@ set(AZURE_CORE_AMQP_SOURCE
     src/network/socket_listener.cpp
     src/network/socket_transport.cpp
     src/network/tls_transport.cpp
-    src/network/transport.cpp 
+    src/network/transport.cpp
+    src/network/private/transport_impl.hpp
+    src/private/package_version.hpp
 )
 
 add_library(azure-core-amqp ${AZURE_CORE_AMQP_SOURCE} ${AZURE_CORE_AMQP_HEADER})

sdk/core/azure-core-amqp/test/ut/CMakeLists.txt

@@ -27,8 +27,9 @@ add_executable(azure-core-amqp-tests
   management_tests.cpp
   message_sender_receiver.cpp
   message_source_target.cpp
+  mock_amqp_server.hpp
   session_tests.cpp
-  transport_tests.cpp 
+  transport_tests.cpp
 )
 add_dependencies(azure-core-amqp-tests Azure::azure-core-amqp)
 

sdk/core/azure-core-amqp/test/ut_uamqp/CMakeLists.txt

@@ -27,7 +27,9 @@ find_package(azure_c_shared_utility CONFIG REQUIRED)
 # Unit Tests
 add_executable(azure-core-amqp-uamqp-tests
   amqp_error_tests.cpp
- "amqp_insertion_tests.cpp")
+  amqp_insertion_tests.cpp
+)
+
 add_dependencies(azure-core-amqp-uamqp-tests Azure::azure-core-amqp uamqp)
 
 # UAMQP Definitions and headers.

sdk/core/azure-core-test/CMakeLists.txt

@@ -14,14 +14,15 @@ set(
   inc/azure/core/test/test_base.hpp
   inc/azure/core/test/test_context_manager.hpp
   inc/azure/core/test/test_proxy_manager.hpp
+  inc/azure/core/test/test_proxy_policy.hpp
 )
 
 set(
   AZURE_CORE_TEST_SOURCE
-  src/private/package_version.hpp
-  src/test_proxy_policy.cpp
   src/test_base.cpp
   src/test_proxy_manager.cpp
+  src/test_proxy_policy.cpp
+  src/private/package_version.hpp
 )
 
 add_library (

sdk/core/azure-core-tracing-opentelemetry/CMakeLists.txt

@@ -40,14 +40,17 @@ if (BUILD_AZURE_CORE_TRACING_OPENTELEMETRY)
 
     set(
       AZURE_CORE_OPENTELEMETRY_HEADER
-        inc/azure/core/tracing/opentelemetry/internal/apiview.hpp
+        inc/azure/core/tracing/opentelemetry/dll_import_export.hpp
         inc/azure/core/tracing/opentelemetry/opentelemetry.hpp
-        src/opentelemetry_private.hpp
+        inc/azure/core/tracing/opentelemetry/rtti.hpp
+        inc/azure/core/tracing/opentelemetry/internal/apiview.hpp
     )
 
     set(
       AZURE_CORE_OPENTELEMETRY_SOURCE
-      src/opentelemetry.cpp
+        src/opentelemetry_private.hpp
+        src/opentelemetry.cpp
+        src/private/package_version.hpp
     )
 
     add_library(azure-core-tracing-opentelemetry ${AZURE_CORE_OPENTELEMETRY_HEADER} ${AZURE_CORE_OPENTELEMETRY_SOURCE})

sdk/core/azure-core-tracing-opentelemetry/test/ut/CMakeLists.txt

@@ -29,7 +29,10 @@ include(GoogleTest)
 add_executable (
   azure-core-tracing-opentelemetry-test
     azure_core_otel_test.cpp
-    azure_core_test.cpp service_support_test.cpp)
+    azure_core_test.cpp
+    service_support_test.cpp
+    test_exporter.hpp
+)
 
 if (MSVC)
   # Disable warnings:

sdk/core/azure-core/CMakeLists.txt

@@ -62,54 +62,56 @@ set(
   AZURE_CORE_HEADER
     ${CURL_TRANSPORT_ADAPTER_INC}
     ${WIN_TRANSPORT_ADAPTER_INC}
-    inc/azure/core.hpp 
+    inc/azure/core.hpp
     inc/azure/core/azure_assert.hpp
     inc/azure/core/base64.hpp
     inc/azure/core/case_insensitive_containers.hpp
     inc/azure/core/context.hpp
-    inc/azure/core/credentials/credentials.hpp
-    inc/azure/core/credentials/token_credential_options.hpp
-    inc/azure/core/cryptography/hash.hpp
     inc/azure/core/datetime.hpp
-    inc/azure/core/diagnostics/logger.hpp
     inc/azure/core/dll_import_export.hpp
     inc/azure/core/etag.hpp
     inc/azure/core/exception.hpp
-    inc/azure/core/http/http.hpp
+    inc/azure/core/match_conditions.hpp
+    inc/azure/core/modified_conditions.hpp
+    inc/azure/core/nullable.hpp
+    inc/azure/core/operation_status.hpp
+    inc/azure/core/operation.hpp
+    inc/azure/core/paged_response.hpp
+    inc/azure/core/platform.hpp
+    inc/azure/core/response.hpp
+    inc/azure/core/rtti.hpp
+    inc/azure/core/url.hpp
+    inc/azure/core/uuid.hpp
+    inc/azure/core/credentials/credentials.hpp
+    inc/azure/core/credentials/token_credential_options.hpp
+    inc/azure/core/cryptography/hash.hpp
+    inc/azure/core/diagnostics/logger.hpp
     inc/azure/core/http/http_status_code.hpp
-    inc/azure/core/http/policies/policy.hpp
+    inc/azure/core/http/http.hpp
     inc/azure/core/http/raw_response.hpp
     inc/azure/core/http/transport.hpp
-    inc/azure/core/internal/credentials/authorization_challenge_parser.hpp
+    inc/azure/core/http/policies/policy.hpp
     inc/azure/core/internal/client_options.hpp
     inc/azure/core/internal/contract.hpp
-    inc/azure/core/internal/cryptography/sha_hash.hpp
-    inc/azure/core/internal/diagnostics/log.hpp
     inc/azure/core/internal/environment.hpp
     inc/azure/core/internal/extendable_enumeration.hpp
+    inc/azure/core/internal/strings.hpp
+    inc/azure/core/internal/unique_handle.hpp
+    inc/azure/core/internal/credentials/authorization_challenge_parser.hpp
+    inc/azure/core/internal/cryptography/sha_hash.hpp
+    inc/azure/core/internal/diagnostics/global_exception.hpp
+    inc/azure/core/internal/diagnostics/log.hpp
     inc/azure/core/internal/http/http_sanitizer.hpp
     inc/azure/core/internal/http/pipeline.hpp
     inc/azure/core/internal/http/user_agent.hpp
     inc/azure/core/internal/io/null_body_stream.hpp
-    inc/azure/core/internal/json/json.hpp
     inc/azure/core/internal/json/json_optional.hpp
     inc/azure/core/internal/json/json_serializable.hpp
-    inc/azure/core/internal/strings.hpp
+    inc/azure/core/internal/json/json.hpp
     inc/azure/core/internal/tracing/service_tracing.hpp
     inc/azure/core/internal/tracing/tracing_impl.hpp
     inc/azure/core/io/body_stream.hpp
-    inc/azure/core/match_conditions.hpp
-    inc/azure/core/modified_conditions.hpp
-    inc/azure/core/nullable.hpp
-    inc/azure/core/operation.hpp
-    inc/azure/core/operation_status.hpp
-    inc/azure/core/paged_response.hpp
-    inc/azure/core/platform.hpp
-    inc/azure/core/response.hpp
-    inc/azure/core/rtti.hpp
     inc/azure/core/tracing/tracing.hpp
-    inc/azure/core/url.hpp
-    inc/azure/core/uuid.hpp
     )
 
 set(
@@ -119,35 +121,35 @@ set(
     src/azure_assert.cpp
     src/base64.cpp
     src/context.cpp
-    src/credentials/authorization_challenge_parser.cpp
-    src/cryptography/md5.cpp
-    src/cryptography/sha_hash.cpp
     src/datetime.cpp
-    src/environment.cpp
     src/environment_log_level_listener.cpp
+    src/environment.cpp
     src/etag.cpp
     src/exception.cpp
+    src/logger.cpp
+    src/operation_status.cpp
+    src/uuid.cpp
+    src/credentials/authorization_challenge_parser.cpp
+    src/cryptography/md5.cpp
+    src/cryptography/sha_hash.cpp
     src/http/bearer_token_authentication_policy.cpp
-    src/http/http.cpp
     src/http/http_sanitizer.cpp
+    src/http/http.cpp
     src/http/log_policy.cpp
     src/http/policy.cpp
     src/http/raw_response.cpp
-    src/http/request.cpp
     src/http/request_activity_policy.cpp
+    src/http/request.cpp
     src/http/retry_policy.cpp
     src/http/telemetry_policy.cpp
     src/http/transport_policy.cpp
     src/http/url.cpp
     src/http/user_agent.cpp
     src/io/body_stream.cpp
     src/io/random_access_file_body_stream.cpp
-    src/logger.cpp
-    src/operation_status.cpp
     src/private/environment_log_level_listener.hpp
     src/private/package_version.hpp
     src/tracing/tracing.cpp
-    src/uuid.cpp
     )
 
 add_library(azure-core ${AZURE_CORE_HEADER} ${AZURE_CORE_SOURCE})

sdk/core/azure-core/test/perf/CMakeLists.txt

@@ -10,6 +10,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED True)
 set(
   AZURE_CORE_PERF_TEST_HEADER
   inc/azure/core/test/nullable_test.hpp
+  inc/azure/core/test/uuid_test.hpp
 )
 
 set(