Skip to content

Commit 844221b

Browse files
committed
Fix compilation errors in tests.
1 parent 29f703b commit 844221b

File tree

4 files changed

+9
-10
lines changed

4 files changed

+9
-10
lines changed

graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/datatype/PRangeTests.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,13 +90,15 @@ public void loopWithStep() throws UnexpectedResultException {
9090
@Test
9191
public void getItem() {
9292
PRange range = PythonObjectFactory.create().createRange(10);
93-
assertEquals(3, range.getItem(3));
93+
assertEquals(3, range.getItemNormalized(3));
9494
}
9595

9696
@Test
9797
public void getItemNegative() {
9898
PRange range = PythonObjectFactory.create().createRange(10);
99-
assertEquals(7, range.getItem(-3));
99+
assertEquals(7, range.getItemNormalized(-3));
100+
String source = "print(range(10)[-3])\n";
101+
assertPrints("7\n", source);
100102
}
101103

102104
@Test

graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/grammar/TestParserTranslator.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -182,11 +182,11 @@ public void parseNumbers() {
182182
public void parseLiteralList() {
183183
PList list = literalAs("[1, 2]", ListLiteralNode.class, PList.class);
184184
assertEquals(2, list.len());
185-
assertEquals(1, list.getItem(0));
186-
assertEquals(2, list.getItem(1));
185+
assertEquals(1, list.getSequenceStorage().getItemNormalized(0));
186+
assertEquals(2, list.getSequenceStorage().getItemNormalized(1));
187187
list = literalAs("[1]", ListLiteralNode.class, PList.class);
188188
assertEquals(1, list.len());
189-
assertEquals(1, list.getItem(0));
189+
assertEquals(1, list.getSequenceStorage().getItemNormalized(0));
190190
list = literalAs("[]", ListLiteralNode.class, PList.class);
191191
assertEquals(0, list.len());
192192
}

graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/runtime/SequenceStorageTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public void objectsGetAndSet() {
4848
@Test
4949
public void objectsGetSlice() {
5050
ObjectSequenceStorage store = new ObjectSequenceStorage(getObjectValues());
51-
ObjectSequenceStorage slice = (ObjectSequenceStorage) store.getSliceInBound(1, 4, 1, 3);
51+
ObjectSequenceStorage slice = store.getSliceInBound(1, 4, 1, 3);
5252

5353
for (int i = 0; i < 3; i++) {
5454
assertEquals(i + 2, slice.getItemNormalized(i));
@@ -140,7 +140,7 @@ public void intGetAndSet() throws SequenceStoreException {
140140
@Test
141141
public void intGetSlice() {
142142
IntSequenceStorage store = new IntSequenceStorage(getIntValues());
143-
IntSequenceStorage slice = (IntSequenceStorage) store.getSliceInBound(1, 4, 1, 3);
143+
IntSequenceStorage slice = store.getSliceInBound(1, 4, 1, 3);
144144

145145
for (int i = 0; i < 3; i++) {
146146
assertEquals(i + 2, slice.getItemNormalized(i));

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/sequence/PSequence.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,13 +38,10 @@ public PSequence(PythonClass cls) {
3838
super(cls);
3939
}
4040

41-
@Deprecated
4241
public abstract void setSlice(int start, int stop, int step, PSequence value);
4342

44-
@Deprecated
4543
public abstract void setSlice(PSlice slice, PSequence value);
4644

47-
@Deprecated
4845
public abstract void delItem(int idx);
4946

5047
public abstract SequenceStorage getSequenceStorage();

0 commit comments

Comments
 (0)