fix(tcp): add missing feature to test

[thvdveld]

#867 was merged with failing tests. Somehow, github allowed this. I thought the merge queue was going to prevent that.

[codecov]

Codecov Report

Attention: 41 lines in your changes are missing coverage. Please review.

Comparison is base (73ac7a4) 79.58% compared to head (080ca4f) 79.74%.
Report is 13 commits behind head on main.

Files	Patch %	Lines
src/wire/ieee802154.rs	73.57%	37 Missing ⚠️
src/iface/interface/sixlowpan.rs	77.77%	2 Missing ⚠️
src/iface/interface/ipv6.rs	88.88%	1 Missing ⚠️
src/iface/interface/mod.rs	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #872      +/-   ##
==========================================
+ Coverage   79.58%   79.74%   +0.16%     
==========================================
  Files          78       78              
  Lines       28122    28137      +15     
==========================================
+ Hits        22380    22439      +59     
+ Misses       5742     5698      -44     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[thvdveld]

@Dirbaio could you check in branch protection if "Only merge non-failing pull requests" is checked for merge queues?

[Dirbaio]

no idea what happened. required tests are configured like this

and the tests job depends on all tests. it shows as "skipped", no idea how could github consider it success...??

[thvdveld]

There should be a checkmark for "Require merge queue" I think.

https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-protected-branches/managing-a-branch-protection-rule#creating-a-branch-protection-rule item 11

[Dirbaio]

yes, that's checked :|