[services] Tunnelbroker - Fix Dockerfile to run unit tests
AbandonedPublic
Actions

Authored by • max on Aug 5 2022, 4:16 AM.

Details

Reviewers

• karol
tomek

Summary

As a part of the diffs stack to prepare running the unit tests in CI this diff fixes the Dockerfile which always runs the service instead of running tests while the COMM_TEST_SERVICES is exported. This bug was fixed in blob and backup, now it's fixed in Tunnelbroker.

The related Linear task: ENG-1491

Test Plan

Running of yarn run-unit-tests tunnelbroker building and start the tests (runTests executable) instead of run service executable.

Diff Detail

Repository

rCOMM Comm

Branch

fix-dockerfile-to-run-tests

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• max created this revision.Aug 5 2022, 4:16 AM

• max held this revision as a draft.

Herald added subscribers: • abosh, • karol, atul and 3 others. · View Herald TranscriptAug 5 2022, 4:16 AM

• max published this revision for review.Aug 5 2022, 4:22 AM

• max retitled this revision from [services] Tunnelbroker - Fix Dockerfile to run unit tests script to [services] Tunnelbroker - Fix Dockerfile to run unit tests.

• max edited the summary of this revision. (Show Details)

• max edited the test plan for this revision. (Show Details)

• max added reviewers: • karol, tomek.

Harbormaster completed remote builds in B11148: Diff 15351.Aug 5 2022, 4:29 AM

• max added a child revision: D4768: [services] Tunnelbroker - Amqp Manager in parallel threads messages throughput test.Aug 6 2022, 1:06 PM

Duplicate of D4052

Still not answered: https://phab.comm.dev/D4052#114760

This revision now requires changes to proceed.Aug 8 2022, 4:59 AM

Abandon because of D4052

• max mentioned this in D4052: [services] Tunnelbroker - Fix scripts in Dockerfiles.Aug 8 2022, 12:05 PM

• max removed a child revision: D4768: [services] Tunnelbroker - Amqp Manager in parallel threads messages throughput test.Aug 24 2022, 5:35 AM