Skip to content

tests/gcoap_dns: Remove duplicate #19154

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

Merged
merged 1 commit into from
Jan 16, 2023
Merged

Conversation

Teufelchen1
Copy link
Contributor

Contribution description

Removes a duplicated test. Originally, the test should have looked different and not be a duplicate. However, the behavior the test aimed at is no longer present anyway, hence it can be removed - rather than fixed.

Further insights can be provided by @miri64

@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Jan 16, 2023
@benpicco benpicco requested a review from miri64 January 16, 2023 10:25
@benpicco benpicco added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jan 16, 2023
Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

bors merge

@bors
Copy link
Contributor

bors bot commented Jan 16, 2023

🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

@riot-ci
Copy link

riot-ci commented Jan 16, 2023

Murdock results

✔️ PASSED

c175213 tests/gcoap_dns: Remove duplicate

Success Failures Total Runtime
13 0 13 01m:19s

Artifacts

@miri64
Copy link
Member

miri64 commented Jan 16, 2023

bors merge

bors bot added a commit that referenced this pull request Jan 16, 2023
19154: tests/gcoap_dns: Remove duplicate r=miri64 a=Teufelchen1

### Contribution description

Removes a duplicated test. Originally, the test should have looked different and not be a duplicate. However, the behavior the test aimed at is no longer present anyway, hence it can be removed - rather than fixed. 

Further insights can be provided by `@miri64` 

Co-authored-by: Teufelchen1 <bennet.blischke@haw-hamburg.de>
@miri64
Copy link
Member

miri64 commented Jan 16, 2023

bors cancel
bors merge

@bors
Copy link
Contributor

bors bot commented Jan 16, 2023

Canceled.

bors bot added a commit that referenced this pull request Jan 16, 2023
18863: boards/esp32s2-mini: add definition for ESP32 S2 Mini r=miri64 a=benpicco



18934: shell/gnrc_icmpv6_echo: separate ICMPv6 echo sending / parsing from shell command r=miri64 a=benpicco



19154: tests/gcoap_dns: Remove duplicate r=miri64 a=Teufelchen1

### Contribution description

Removes a duplicated test. Originally, the test should have looked different and not be a duplicate. However, the behavior the test aimed at is no longer present anyway, hence it can be removed - rather than fixed. 

Further insights can be provided by `@miri64` 

Co-authored-by: Benjamin Valentin <benjamin.valentin@bht-berlin.de>
Co-authored-by: Benjamin Valentin <benjamin.valentin@ml-pa.com>
Co-authored-by: Teufelchen1 <bennet.blischke@haw-hamburg.de>
@bors
Copy link
Contributor

bors bot commented Jan 16, 2023

Build failed (retrying...):

bors bot added a commit that referenced this pull request Jan 16, 2023
18934: shell/gnrc_icmpv6_echo: separate ICMPv6 echo sending / parsing from shell command r=miri64 a=benpicco



19060: SUBSYSTEMS.md: migrate subsystem list from wiki r=maribu a=jia200x



19154: tests/gcoap_dns: Remove duplicate r=miri64 a=Teufelchen1

### Contribution description

Removes a duplicated test. Originally, the test should have looked different and not be a duplicate. However, the behavior the test aimed at is no longer present anyway, hence it can be removed - rather than fixed. 

Further insights can be provided by `@miri64` 

Co-authored-by: Benjamin Valentin <benjamin.valentin@bht-berlin.de>
Co-authored-by: Jose Alamos <jose@alamos.cc>
Co-authored-by: Teufelchen1 <bennet.blischke@haw-hamburg.de>
@bors
Copy link
Contributor

bors bot commented Jan 16, 2023

This PR was included in a batch that was canceled, it will be automatically retried

bors bot added a commit that referenced this pull request Jan 16, 2023
18934: shell/gnrc_icmpv6_echo: separate ICMPv6 echo sending / parsing from shell command r=maribu a=benpicco



19060: SUBSYSTEMS.md: migrate subsystem list from wiki r=maribu a=jia200x



19154: tests/gcoap_dns: Remove duplicate r=miri64 a=Teufelchen1

### Contribution description

Removes a duplicated test. Originally, the test should have looked different and not be a duplicate. However, the behavior the test aimed at is no longer present anyway, hence it can be removed - rather than fixed. 

Further insights can be provided by `@miri64` 

Co-authored-by: Benjamin Valentin <benjamin.valentin@bht-berlin.de>
Co-authored-by: Jose Alamos <jose@alamos.cc>
Co-authored-by: Teufelchen1 <bennet.blischke@haw-hamburg.de>
@bors
Copy link
Contributor

bors bot commented Jan 16, 2023

Build failed (retrying...):

@bors
Copy link
Contributor

bors bot commented Jan 16, 2023

Build succeeded:

@bors bors bot merged commit a3bfdca into RIOT-OS:master Jan 16, 2023
@jia200x jia200x added this to the Release 2023.04 milestone Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants