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

add libuv and cmocka test cases with file system on sim and qemu on CI #11966

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

vela-mib
Copy link
Contributor

@vela-mib vela-mib commented Mar 21, 2024

Summary

add libuv and cmocka test cases with file system on sim and qemu on CI

Impact

CI run

Testing

localhost

@acassis
Copy link
Contributor

acassis commented Mar 21, 2024

Please update this page: https://nuttx.apache.org/docs/latest/platforms/risc-v/qemu-rv/boards/rv-virt/index.html to include these new tests. Please include the steps to run these python scripts, the qemu parameters, etc

@vela-mib
Copy link
Contributor Author

Please update this page: https://nuttx.apache.org/docs/latest/platforms/risc-v/qemu-rv/boards/rv-virt/index.html to include these new tests. Please include the steps to run these python scripts, the qemu parameters, etc

done.

@acassis
Copy link
Contributor

acassis commented Mar 22, 2024

Please fix these errors:

kernel/fs/cases/fs_eventfd_test.c: In function 'thread_func':
Error: kernel/fs/cases/fs_eventfd_test.c:50:12: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
   50 |       read((int)args, &eventfd01_buffer, sizeof(eventfd_t));
      |            ^
cc1: all warnings being treated as errors
make[2]: *** [/github/workspace/sources/apps/Application.mk:200: kernel/fs/cases/fs_eventfd_test.c.github.workspace.sources.apps.testing.testsuites.o] Error 1
kernel/fs/cases/fs_fsync_test.c: In function 'test_nuttx_fs_fsync02':
Error: kernel/fs/cases/fs_fsync_test.c:120:34: error: format '%d' expects argument of type 'int', but argument 3 has type 'size_t' {aka 'long unsigned int'} [-Werror=format=]
  120 |   syslog(LOG_INFO, "the bsize = %d\n", statfsbuf.f_bsize);
      |                                 ~^     ~~~~~~~~~~~~~~~~~
      |                                  |              |
      |                                  int            size_t {aka long unsigned int}
      |                                 %ld
cc1: all warnings being treated as errors
make[2]: *** [/github/workspace/sources/apps/Application.mk:200: kernel/fs/cases/fs_fsync_test.c.github.workspace.sources.apps.testing.testsuites.o] Error 1
kernel/fs/cases/fs_stream_test.c: In function 'test_nuttx_fs_stream03':
Error: kernel/fs/cases/fs_stream_test.c:270:61: error: format '%zi' expects argument of type 'signed size_t', but argument 4 has type 'int' [-Werror=format=]
  270 |                  "len = %zi != return value from fwrite = %zi",
      |                                                           ~~^
      |                                                             |
      |                                                             long int
      |                                                           %i
  271 |                  len, ret);
      |                       ~~~                                    
      |                       |
      |                       int
Error: kernel/fs/cases/fs_stream_test.c:303:60: error: format '%zi' expects argument of type 'signed size_t', but argument 4 has type 'int' [-Werror=format=]
  303 |                  "len = %zi != return value from fread = %zi",
      |                                                          ~~^
      |                                                            |
      |                                                            long int
      |                                                          %i
  304 |                  len, ret);
      |                       ~~~                                   
      |                       |
      |                       int
cc1: all warnings being treated as errors
make[2]: *** [/github/workspace/sources/apps/Application.mk:200: kernel/fs/cases/fs_stream_test.c.github.workspace.sources.apps.testing.testsuites.o] Error 1
kernel/time/cases/clock_test_timer03.c: In function 'test_nuttx_clock_test_timer03':
Error: kernel/time/cases/clock_test_timer03.c:126:39: error: format '%lld' expects argument of type 'long long int', but argument 3 has type 'int64_t' {aka 'long int'} [-Werror=format=]
  126 |       syslog(LOG_INFO, "expected = %lld escaped = %lld failed = %d",
      |                                    ~~~^
      |                                       |
      |                                       long long int
      |                                    %ld
  127 |              expected, escaped, failed);
      |              ~~~~~~~~                  
      |              |
      |              int64_t {aka long int}
Error: kernel/time/cases/clock_test_timer03.c:126:54: error: format '%lld' expects argument of type 'long long int', but argument 4 has type 'int64_t' {aka 'long int'} [-Werror=format=]
  126 |       syslog(LOG_INFO, "expected = %lld escaped = %lld failed = %d",
      |                                                   ~~~^
      |                                                      |
      |                                                      long long int
      |                                                   %ld
  127 |              expected, escaped, failed);
      |                        ~~~~~~~                        
      |                        |
      |                        int64_t {aka long int}
cc1: all warnings being treated as errors

@vela-mib vela-mib force-pushed the master branch 5 times, most recently from 919cc63 to c64f964 Compare March 27, 2024 07:10
@vela-mib vela-mib force-pushed the master branch 3 times, most recently from d927e9c to efc00ea Compare April 3, 2024 01:35
@xiaoxiang781216 xiaoxiang781216 merged commit 9847c62 into apache:master Apr 10, 2024
26 checks passed
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.

3 participants