Refactor trace exporter to utilize enrollment details from knapsack

[cesarfda]

This pull request includes significant changes to the TraceExporter class and related tests, focusing on removing the dependency on the osqueryClient and refactoring the way attributes are added from enrollment details.

Resolves #2076

Refactoring TraceExporter:

• Removed the osqueryClient dependency from TraceExporter and related methods. (pkg/traces/exporter/exporter.go) [1] [2]
• Changed the method addAttributesFromOsquery to use enrollment details instead of querying osquery. (pkg/traces/exporter/exporter.go)
• Updated the initialization of TraceExporter to call addAttributesFromOsquery directly. (pkg/traces/exporter/exporter.go)

Test Updates:

• Removed mocks and expectations related to osqueryClient from tests. (pkg/traces/exporter/exporter_test.go) [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]
• Updated test cases to use GetEnrollmentDetails for providing necessary attributes. (pkg/traces/exporter/exporter_test.go) [1] [2] [3]

Minor Cleanup:

• Removed unused imports and variables. (pkg/traces/exporter/exporter.go) [1] [2] [3]
• Removed the SetOsqueryClient method and related calls. (pkg/traces/exporter/exporter.go)

These changes streamline the TraceExporter class by eliminating the need for an osqueryClient and leveraging enrollment details directly, simplifying the code and improving maintainability.

[RebeccaMahany]

Nice! Looks much simpler

[RebeccaMahany]

LGTM!