Skip to content

Commit

Permalink
adjust test socket close down and pull of zephyr sdk
Browse files Browse the repository at this point in the history
  • Loading branch information
JacobBarthelmeh committed Jan 29, 2024
1 parent 126a884 commit 7156ac8
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/zephyr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,10 @@ jobs:

- name: Install zephyr SDK
run: |
wget -q https://github.com/zephyrproject-rtos/sdk-ng/releases/download/v${{ matrix.config.zephyr-sdk }}/zephyr-sdk-${{ matrix.config.zephyr-sdk }}_linux-x86_64.tar.xz
tar xf zephyr-sdk-${{ matrix.config.zephyr-sdk }}_linux-x86_64.tar.xz
wget -q https://github.com/zephyrproject-rtos/sdk-ng/releases/download/v${{ matrix.config.zephyr-sdk }}/zephyr-sdk-${{ matrix.config.zephyr-sdk }}_linux-x86_64_minimal.tar.xz
tar xf zephyr-sdk-${{ matrix.config.zephyr-sdk }}_linux-x86_64_minimal.tar.xz
cd zephyr-sdk-${{ matrix.config.zephyr-sdk }}
./setup.sh -h -c
./setup.sh -h -c -t x86_64-zephyr-elf
- name: Run wolfssh tests
id: wolfssh-test
Expand Down
5 changes: 5 additions & 0 deletions tests/api.c
Original file line number Diff line number Diff line change
Expand Up @@ -953,6 +953,7 @@ static void test_wolfSSH_SFTP_SendReadPacket(void)
func_args ser;
tcp_ready ready;
int argsCount;
int clientFd;

const char* args[10];
WOLFSSH_CTX* ctx = NULL;
Expand Down Expand Up @@ -1061,6 +1062,10 @@ static void test_wolfSSH_SFTP_SendReadPacket(void)

AssertIntEQ(argsCount, WS_SUCCESS);

/* close client socket down */
clientFd = wolfSSH_get_fd(ssh);
close(clientFd);

wolfSSH_free(ssh);
wolfSSH_CTX_free(ctx);
#ifdef WOLFSSH_ZEPHYR
Expand Down
2 changes: 1 addition & 1 deletion zephyr/samples/tests/sample.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ common:
- "Zephyr wolfSSH tests passed"
tests:
sample.lib.wolfssh_tests:
timeout: 50
timeout: 200
platform_allow: qemu_x86
integration_platforms:
- qemu_x86

0 comments on commit 7156ac8

Please sign in to comment.