Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fail to run the test driver of devenv #133

Open
yuhow opened this issue Jan 9, 2023 · 1 comment
Open

Fail to run the test driver of devenv #133

yuhow opened this issue Jan 9, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@yuhow
Copy link
Contributor

yuhow commented Jan 9, 2023

The output message are listed below:

*** test file: test_init_var.sh
test_devenvroot
test_devenvflavor
test_devenvflavorroot
test_devenvcurrentroot
test_devenvlibrary_path_backup
test_devenvpath_backup
test_devenvprefix
test_devenvdlroot

Ran 8 tests.

OK
*** test file: test_bash_utils.sh
test_display_type
ASSERT:expected: but was:<>
shunit2:ERROR test_display_type() returned non-zero return code.
test_display
test_display_e
devenv shunit2 unittest
test_get_cmd_list
test_get_build_list
test_get_application_list
test_get_flavor_list

Ran 7 tests.

FAILED (failures=2)
*** test file: test_devenv_impl.sh
test_namemunge
test_nameremove

Ran 2 tests.

OK


There is a conflict between the function 'display()' of devenv and the shell command /usr/bin/display.
The display is a command provided by the package 'imagemagick'.
Possibly we should change the function name of 'display()' of devenv.

@yungyuc yungyuc added the enhancement New feature or request label Jan 9, 2023
@yungyuc
Copy link
Member

yungyuc commented Jan 9, 2023

Agree. display is a very generic name and the build system should yield. If we cannot think of a better verb for the helper, perhaps a prefix like devenv_display can be used.

@yungyuc yungyuc changed the title Fail to run the test driver of devenc Fail to run the test driver of devenv Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants