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

Automatic time update/NITZ unit tests #8

Merged
merged 5 commits into from Jan 31, 2013

Conversation

pgerdt
Copy link

@pgerdt pgerdt commented Jan 28, 2013

Unit tests for "public" classes included in the handling of automatic time/timezone update: NetworkTimeInfo, NetworkTime, NetworkOperator.

In addition, adds .gitignore

@pgerdt
Copy link
Author

pgerdt commented Jan 28, 2013

Updated pull request: added missing files.


#define log_debug(msg, ...) \
do { \
printf("log_debug: "); printf(msg, ##__VA_ARGS__); printf("\n"); \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor, but If the msg is always literal string, then you could do just: printf("log_debug: " msg "\n", ##VA_ARGS);

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will update, looks better!

@pgerdt
Copy link
Author

pgerdt commented Jan 29, 2013

Updated pull request, changes:

  • Updated qmlogstub.h prinf syntax as suggested by spiiroin [1]
  • Cleaned up includes in all (!) files expect gmlogstub.h in tests/ut_networktime
  • Changed test installation directory to /opt/tests/timed-tests
  • Updated tests.xml and changed its installation dir to /opt/tests/timed-tests/test-definition
  • Added top-level pro file + renamend src/root.pro to src/src.pro for a smoother build experience

[1] spiiroin about tests/ut_networktime/qmlogstub.h: Minor, but If the msg is always literal string, then you could do just: printf("log_debug: " msg "\n", ##VA_ARGS);

@pgerdt
Copy link
Author

pgerdt commented Jan 29, 2013

The changes in this pull request obviously require changes to the spec file used in build.pub.meego.com timed OBS project.

Here's a OBS branch of timed that includes changes in this pull request and has an updated spec file.

Petri M. Gerdt added 4 commits January 29, 2013 16:19
Move tests.xml to test directory, add automatic time update
related unit tests, remove obsolete tests.
Test setup changes: test.xml is installed to directory
/opt/tests/timed-tests/test-definition.
Rename src/root.pro to src/src.pro for a smoother build setup.
@pgerdt
Copy link
Author

pgerdt commented Jan 29, 2013

Updated pull request, added copyright headers.

pgerdt pushed a commit that referenced this pull request Jan 31, 2013
Automatic time update/NITZ unit tests
@pgerdt pgerdt merged commit f92b08c into nemomobile:master Jan 31, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants