From 762c77eeb018d2d434017dab46dd509d42a67b64 Mon Sep 17 00:00:00 2001 From: Ben Chatelain Date: Sun, 11 Feb 2018 16:23:48 -0700 Subject: [PATCH] Add caveat about coverage report --- README.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index 2edc1c8..6bc4da6 100644 --- a/README.md +++ b/README.md @@ -132,12 +132,13 @@ Build output can be found in the `build/` directory within the project. ## ๐Ÿšงโœ… Tests -The tests in this project are a work-in-progress. Since Xcode does not officially support tests for command-line tool targets, there are some manual actions necessary to create and/or update the tests: +The tests in this project are a work-in-progress. Since Xcode does not officially support tests for command-line tool targets, there is some strange behavior and manual actions necessary to create and/or update the tests: - Types from the `mas` target must be included in the `mas-tests` target in order to be used in a test. - `@testable import mas` does not work - XCTest is the current test framework - this may change in the future to Quick/Nimble. +- Code coverage doesn't show up for code under test until you enable "Show Test Bundles". Presumably, this is because production code is currently being added to two targets and Xcode is getting confused. We may move the app code into a framework target to make it easier to test.