Skip to content

Rewrite bash tests in go#717

Closed
AzizMukhtorjonov wants to merge 25 commits into
zigbee-alliance:masterfrom
AzizMukhtorjonov:azizmukhtorjonov/rewrite-tests-to-go
Closed

Rewrite bash tests in go#717
AzizMukhtorjonov wants to merge 25 commits into
zigbee-alliance:masterfrom
AzizMukhtorjonov:azizmukhtorjonov/rewrite-tests-to-go

Conversation

@AzizMukhtorjonov

@AzizMukhtorjonov AzizMukhtorjonov commented Apr 13, 2026

Copy link
Copy Markdown
Collaborator
  • Rewrite bash CLI integration tests to go tests
  • Remove migrated bash tests
  • Refactor setting up pool for integration tests

@Abdulbois Abdulbois changed the title Rewrite bash tests in go Draft: Rewrite bash tests in go Apr 13, 2026
@Abdulbois Abdulbois marked this pull request as draft April 13, 2026 16:18
@Abdulbois Abdulbois changed the title Draft: Rewrite bash tests in go Rewrite bash tests in go Apr 13, 2026
Comment thread Dockerfile Outdated
############################
FROM ubuntu:20.04 AS builder

ARG TARGETARCH=amd64

@Abdulbois Abdulbois May 11, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why it's needed? I mean supporting other archs should be made as separate task

Comment thread integration_tests/cli/upgrade-demo.sh Outdated
get_height current_height
echo "Current height is $current_height"
plan_height=$(expr $current_height + 3)
plan_height=$(expr $current_height + 30)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why its changed?

@Abdulbois

Copy link
Copy Markdown
Collaborator

Please enable running rewritten tests in CI(verify.yml).

Also, ensure that the coverage job is working.

AzizMukhtorjonov and others added 21 commits May 18, 2026 11:20
…to-go

# Conflicts:
#	Makefile
#	Makefile.test
#	genlocalnetappbins.sh
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
Signed-off-by: Abdulbois <abdulbois.tursunov@dsr-corporation.com>
@Abdulbois Abdulbois marked this pull request as ready for review May 25, 2026 08:19
@AzizMukhtorjonov AzizMukhtorjonov closed this by deleting the head repository May 25, 2026
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.

2 participants