Skip to content

Commit

Permalink
[test][refactor][style] Port LD-logger tests to python
Browse files Browse the repository at this point in the history
Also adds some failing tests, documenting existing behavior.
  • Loading branch information
steakhal authored and Balazs Benics committed Jan 21, 2022
1 parent 66bfc6e commit 5f7ab80
Show file tree
Hide file tree
Showing 12 changed files with 1,369 additions and 291 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ jobs:
- name: Run build-logger tests
working-directory: analyzer/tools/build-logger
run: |
make -f Makefile.manual
make -f Makefile.manual test
pip install -r requirements_py/dev/requirements.txt
make test
- name: Run merge-clang-extdef-mappings tests
working-directory: analyzer/tools/merge_clang_extdef_mappings
Expand Down
4 changes: 2 additions & 2 deletions analyzer/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -147,9 +147,9 @@ endef
export LOGGER_BUILD_ERROR_MSG
build_ld_logger:
ifeq ($(BUILD_LOGGER_64_BIT_ONLY),YES)
$(MAKE) -C tools/build-logger -f Makefile.manual pack64bit_only 2> /dev/null
$(MAKE) -C $(CC_ANALYZER)/tools/build-logger pack64bit_only 2> /dev/null
else
$(MAKE) -C tools/build-logger -f Makefile.manual 2> /dev/null || (echo "$$LOGGER_BUILD_ERROR_MSG" && false)
$(MAKE) -C $(CC_ANALYZER)/tools/build-logger all 2> /dev/null || (echo "$$LOGGER_BUILD_ERROR_MSG" && false)
endif

# NOTE: extra spaces are allowed and ignored at the beginning of the
Expand Down
2 changes: 1 addition & 1 deletion analyzer/tests/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ test_functional_in_env: venv_dev
$(ACTIVATE_DEV_VENV) && $(FUNCTIONAL_TEST_CMD)

test_build_logger:
make -C tools/build-logger -f Makefile.manual test
make -C tools/build-logger test

test_tu_collector:
REPO_ROOT=$(REPO_ROOT) make -C $(ROOT)/tools/tu_collector test
Expand Down
Loading

0 comments on commit 5f7ab80

Please sign in to comment.