Skip to content

Commit ef352bd

Browse files
authoredDec 23, 2022
Bugfix in automotive staged enumerators (#3803)
* remove scanner tests from pypy * remove scanner tests from pypy * Fix minor bug in argument passing of staged_test_cases * remove scanner tests from all tests which are not allowed to fail * update * disable scanner tests for all tests which are not allowed to fail * update default values of github CI * try to fix ci * try to fix ci * try to fix ci * try to fix ci Co-authored-by: Nils Weiss <[email protected]>
1 parent 0e88e8f commit ef352bd

File tree

2 files changed

+22
-14
lines changed

2 files changed

+22
-14
lines changed
 

‎.github/workflows/unittests.yml

+21-13
Original file line numberDiff line numberDiff line change
@@ -64,67 +64,75 @@ jobs:
6464
name: ${{ matrix.os }} ${{ matrix.installmode }} ${{ matrix.python }} ${{ matrix.mode }} ${{ matrix.flags }}
6565
runs-on: ${{ matrix.os }}
6666
timeout-minutes: 20
67-
continue-on-error: ${{ matrix.allow-failure }}
67+
continue-on-error: ${{ matrix.allow-failure == 'true' }}
6868
strategy:
6969
fail-fast: false
7070
matrix:
7171
os: [ubuntu-latest]
7272
python: ["3.7", "3.8", "3.9", "3.10"]
7373
mode: [non_root]
7474
installmode: ['']
75-
flags: ['']
76-
allow-failure: [false]
75+
flags: [" -K scanner"]
76+
allow-failure: ['false']
7777
include:
7878
# Linux root tests
7979
- os: ubuntu-20.04
8080
python: "2.7"
8181
mode: root
82-
allow-failure: false
82+
flags: " -K scanner"
8383
- os: ubuntu-latest
8484
python: "3.10"
8585
mode: root
86-
allow-failure: false
86+
flags: " -K scanner"
8787
# PyPy tests: root only
8888
- os: ubuntu-20.04
8989
python: "pypy2.7"
9090
mode: root
91-
allow-failure: false
9291
flags: " -K scanner"
9392
- os: ubuntu-latest
9493
python: "pypy3.9"
9594
mode: root
96-
allow-failure: false
9795
flags: " -K scanner"
9896
# Libpcap test
9997
- os: ubuntu-latest
10098
python: "3.10"
10199
mode: root
102100
installmode: 'libpcap'
103-
allow-failure: false
101+
flags: " -K scanner"
104102
# MacOS tests
105103
- os: macos-12
106104
python: "2.7"
107105
mode: both
108-
allow-failure: false
106+
flags: " -K scanner"
109107
- os: macos-12
110108
python: "3.10"
111109
mode: both
112-
allow-failure: false
110+
flags: " -K scanner"
113111
# Scanner tests
112+
- os: ubuntu-20.04
113+
python: "2.7"
114+
mode: root
115+
allow-failure: 'true'
116+
flags: " -k scanner"
117+
- os: ubuntu-latest
118+
python: "3.10"
119+
mode: root
120+
allow-failure: 'true'
121+
flags: " -k scanner"
114122
- os: ubuntu-20.04
115123
python: "pypy2.7"
116124
mode: root
117-
allow-failure: true
125+
allow-failure: 'true'
118126
flags: " -k scanner"
119127
- os: ubuntu-latest
120128
python: "pypy3.9"
121129
mode: root
122-
allow-failure: true
130+
allow-failure: 'true'
123131
flags: " -k scanner"
124132
- os: macos-12
125133
python: "3.10"
126134
mode: both
127-
allow-failure: true
135+
allow-failure: 'true'
128136
flags: " -k scanner"
129137
steps:
130138
- name: Checkout Scapy

‎scapy/contrib/automotive/scanner/staged_test_case.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ def pre_execute(self,
201201

202202
def execute(self, socket, state, **kwargs):
203203
# type: (_SocketUnion, EcuState, Any) -> None
204-
kwargs = self.__current_kwargs or dict()
204+
kwargs.update(self.__current_kwargs or dict())
205205
self.current_test_case.execute(socket, state, **kwargs)
206206

207207
def post_execute(self,

0 commit comments

Comments
 (0)
Please sign in to comment.