amzn / smoke-framework

A light-weight server-side service framework written in the Swift programming language.
Apache License 2.0
1.44k stars 41 forks source link

Missing `SmokeHTTP1` dependency from `SmokeOperationsHTTP1Tests` #77

Closed Davidde94 closed 3 years ago

Davidde94 commented 3 years ago

Non-deterministic, it depends how quickly SPM manages to get to compiling SmokeHTTP. Fix should be really easy, just add the dependency :)

Build log below:

[803/815] Merging module _SmokeOperationsHTTP1Concurrency
[804/815] Merging module SmokeHTTP1
[805/817] Compiling SmokeOperationsHTTP1Server HTTP1OperationTraceContext.swift
/workspace/.logs/**/*.log/5DAB444C-FE1F-49B1-8186-3CEDB8A2AA1D/repos/smoke-framework/Tests/SmokeOperationsHTTP1Tests/SmokeOperationsHTTP1AsyncTests.swift:21:8: error: no such module 'SmokeHTTP1'
import SmokeHTTP1
       ^
[807/828] Compiling SmokeOperationsHTTP1Server JSONPayloadHTTP1OperationDelegate.swift
/workspace/.logs/**/*.log/5DAB444C-FE1F-49B1-8186-3CEDB8A2AA1D/repos/smoke-framework/Tests/SmokeOperationsHTTP1Tests/SmokeOperationsHTTP1AsyncTests.swift:21:8: error: no such module 'SmokeHTTP1'
import SmokeHTTP1
       ^
[810/828] Compiling SmokeOperationsHTTP1Server HTTP1RequestInvocationContext.swift
[811/828] Compiling SmokeOperationsHTTP1Server SmokeServerPerInvocationContextInitializer.swift
[813/828] Compiling SmokeOperationsHTTP1Server SmokeInvocationTraceContext.swift
[814/828] Compiling SmokeOperationsHTTP1Server SmokeHTTP1Server+runAsOperationServer.swift
/workspace/.logs/**/*.log/5DAB444C-FE1F-49B1-8186-3CEDB8A2AA1D/repos/smoke-framework/Tests/SmokeOperationsHTTP1Tests/SmokeOperationsHTTP1AsyncTests.swift:21:8: error: no such module 'SmokeHTTP1'
import SmokeHTTP1
       ^
[817/829] Compiling SmokeOperationsHTTP1Server SmokeServerStaticContextInitializer.swift
[818/829] Compiling SmokeOperationsHTTP1Server SmokeServerInvocationReporting.swift
[819/829] Compiling SmokeOperationsHTTP1Server SmokeServerInvocationContext+HTTP1RequestInvocationContext.swift
/workspace/.logs/**/*.log/5DAB444C-FE1F-49B1-8186-3CEDB8A2AA1D/repos/smoke-framework/Tests/SmokeOperationsHTTP1Tests/SmokeOperationsHTTP1AsyncTests.swift:21:8: error: no such module 'SmokeHTTP1'
import SmokeHTTP1
       ^
Command failed reason=exit, status=1
Davidde94 commented 3 years ago

@tachyonics friendly ping :)

tachyonics commented 3 years ago

Thanks David, I have create a PR for the suggested change. Let me know if I have mis-interpreted what was needed.

Davidde94 commented 3 years ago

@tachyonics thanks for the fix :)