Skip to content

Commit 948960c

Browse files
authored
[Test] Fix - mutateInstance in BulkUpdateApiKeyRequestSerializationTests not unique (elastic#103917)
Leftover `+ 1` from previous testing strategy that should have been removed. Relates: elastic#103913
1 parent a170cd5 commit 948960c

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/security/action/apikey/BulkUpdateApiKeyRequestSerializationTests.java

+1-3
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
package org.elasticsearch.xpack.core.security.action.apikey;
99

10-
import org.apache.lucene.tests.util.LuceneTestCase;
1110
import org.elasticsearch.TransportVersions;
1211
import org.elasticsearch.common.io.stream.Writeable;
1312
import org.elasticsearch.core.TimeValue;
@@ -21,7 +20,6 @@
2120

2221
import static org.hamcrest.Matchers.nullValue;
2322

24-
@LuceneTestCase.AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/103913")
2523
public class BulkUpdateApiKeyRequestSerializationTests extends AbstractWireSerializingTestCase<BulkUpdateApiKeyRequest> {
2624
public void testSerializationBackwardsCompatibility() throws IOException {
2725
BulkUpdateApiKeyRequest testInstance = createTestInstance();
@@ -67,7 +65,7 @@ protected BulkUpdateApiKeyRequest mutateInstance(BulkUpdateApiKeyRequest instanc
6765
instance.getIds(),
6866
instance.getRoleDescriptors(),
6967
metadata,
70-
TimeValue.parseTimeValue((days + 1) + "d", null, "expiration")
68+
TimeValue.parseTimeValue(days + "d", null, "expiration")
7169
);
7270
}
7371
}

0 commit comments

Comments
 (0)