Skip to content

Commit a9792b0

Browse files
committed
Use diverse tool dependencies package index test data
The package indexes for used for projectdata test data had the same packages[].platforms[].toolsDependencies data for every platform. Although there was nothing specifically wrong with this, it seems like diversity in the test data makes the tests more meaningful by differentiating the expected tools dependencies data for each platform.
1 parent 9cd738f commit a9792b0

File tree

3 files changed

+51
-34
lines changed

3 files changed

+51
-34
lines changed

Diff for: internal/project/projectdata/testdata/packageindexes/empty-ids/package_foo_index.json

+22-12
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,12 @@
5252
{
5353
"packager": "arduino",
5454
"name": "avr-gcc",
55-
"version": "4.8.1-arduino5"
55+
"version": "7.3.0-atmel3.6.1-arduino7"
5656
},
5757
{
5858
"packager": "arduino",
5959
"name": "avrdude",
60-
"version": "6.0.1-arduino5"
60+
"version": "6.3.0-arduino17"
6161
}
6262
]
6363
}
@@ -89,13 +89,13 @@
8989
"toolsDependencies": [
9090
{
9191
"packager": "arduino",
92-
"name": "avr-gcc",
93-
"version": "4.8.1-arduino5"
92+
"name": "arm-none-eabi-gcc",
93+
"version": "7-2017q4"
9494
},
9595
{
9696
"packager": "arduino",
97-
"name": "avrdude",
98-
"version": "6.0.1-arduino5"
97+
"name": "bossac",
98+
"version": "1.7.0-arduino3"
9999
}
100100
]
101101
},
@@ -115,13 +115,13 @@
115115
"toolsDependencies": [
116116
{
117117
"packager": "arduino",
118-
"name": "avr-gcc",
119-
"version": "4.8.1-arduino5"
118+
"name": "openocd",
119+
"version": "0.11.0-arduino2"
120120
},
121121
{
122122
"packager": "arduino",
123-
"name": "avrdude",
124-
"version": "6.0.1-arduino5"
123+
"name": "arm-none-eabi-gcc",
124+
"version": "7-2017q4"
125125
}
126126
]
127127
},
@@ -140,14 +140,24 @@
140140
"boards": [{ "name": "" }, { "name": "My Board Pro" }],
141141
"toolsDependencies": [
142142
{
143-
"packager": "arduino",
143+
"packager": "",
144144
"name": "avr-gcc",
145145
"version": "4.8.1-arduino5"
146146
},
147147
{
148148
"packager": "arduino",
149-
"name": "avrdude",
149+
"name": "",
150150
"version": "6.0.1-arduino5"
151+
},
152+
{
153+
"packager": "arduino",
154+
"name": "openocd",
155+
"version": ""
156+
},
157+
{
158+
"packager": "arduino",
159+
"name": "CMSIS",
160+
"version": "4.5.0"
151161
}
152162
]
153163
}

Diff for: internal/project/projectdata/testdata/packageindexes/missing-ids/package_foo_index.json

+19-12
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,12 @@
5151
{
5252
"packager": "arduino",
5353
"name": "avr-gcc",
54-
"version": "4.8.1-arduino5"
54+
"version": "7.3.0-atmel3.6.1-arduino7"
5555
},
5656
{
5757
"packager": "arduino",
5858
"name": "avrdude",
59-
"version": "6.0.1-arduino5"
59+
"version": "6.3.0-arduino17"
6060
}
6161
]
6262
}
@@ -87,13 +87,13 @@
8787
"toolsDependencies": [
8888
{
8989
"packager": "arduino",
90-
"name": "avr-gcc",
91-
"version": "4.8.1-arduino5"
90+
"name": "arm-none-eabi-gcc",
91+
"version": "7-2017q4"
9292
},
9393
{
9494
"packager": "arduino",
95-
"name": "avrdude",
96-
"version": "6.0.1-arduino5"
95+
"name": "bossac",
96+
"version": "1.7.0-arduino3"
9797
}
9898
]
9999
},
@@ -112,13 +112,13 @@
112112
"toolsDependencies": [
113113
{
114114
"packager": "arduino",
115-
"name": "avr-gcc",
116-
"version": "4.8.1-arduino5"
115+
"name": "openocd",
116+
"version": "0.11.0-arduino2"
117117
},
118118
{
119119
"packager": "arduino",
120-
"name": "avrdude",
121-
"version": "6.0.1-arduino5"
120+
"name": "arm-none-eabi-gcc",
121+
"version": "7-2017q4"
122122
}
123123
]
124124
},
@@ -137,14 +137,21 @@
137137
"boards": [{ "foo": "My Board" }, { "name": "My Board Pro" }],
138138
"toolsDependencies": [
139139
{
140-
"packager": "arduino",
141140
"name": "avr-gcc",
142141
"version": "4.8.1-arduino5"
143142
},
144143
{
145144
"packager": "arduino",
146-
"name": "avrdude",
147145
"version": "6.0.1-arduino5"
146+
},
147+
{
148+
"packager": "arduino",
149+
"name": "openocd"
150+
},
151+
{
152+
"packager": "arduino",
153+
"name": "CMSIS",
154+
"version": "4.5.0"
148155
}
149156
]
150157
}

Diff for: internal/project/projectdata/testdata/packageindexes/valid-package-index/package_foo_index.json

+10-10
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,12 @@
5252
{
5353
"packager": "arduino",
5454
"name": "avr-gcc",
55-
"version": "4.8.1-arduino5"
55+
"version": "7.3.0-atmel3.6.1-arduino7"
5656
},
5757
{
5858
"packager": "arduino",
5959
"name": "avrdude",
60-
"version": "6.0.1-arduino5"
60+
"version": "6.3.0-arduino17"
6161
}
6262
]
6363
}
@@ -89,13 +89,13 @@
8989
"toolsDependencies": [
9090
{
9191
"packager": "arduino",
92-
"name": "avr-gcc",
93-
"version": "4.8.1-arduino5"
92+
"name": "arm-none-eabi-gcc",
93+
"version": "7-2017q4"
9494
},
9595
{
9696
"packager": "arduino",
97-
"name": "avrdude",
98-
"version": "6.0.1-arduino5"
97+
"name": "bossac",
98+
"version": "1.7.0-arduino3"
9999
}
100100
]
101101
},
@@ -115,13 +115,13 @@
115115
"toolsDependencies": [
116116
{
117117
"packager": "arduino",
118-
"name": "avr-gcc",
119-
"version": "4.8.1-arduino5"
118+
"name": "openocd",
119+
"version": "0.11.0-arduino2"
120120
},
121121
{
122122
"packager": "arduino",
123-
"name": "avrdude",
124-
"version": "6.0.1-arduino5"
123+
"name": "arm-none-eabi-gcc",
124+
"version": "7-2017q4"
125125
}
126126
]
127127
}

0 commit comments

Comments
 (0)