Skip to content

Commit d6a8a5c

Browse files
committed
Merge branch 'release-3.5.0'
2 parents adf1633 + 004f710 commit d6a8a5c

File tree

21 files changed

+704
-125
lines changed

21 files changed

+704
-125
lines changed

.ci/ci_check.sh

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,6 @@ pwd
159159
ls -la
160160
export JAVA_TOOL_OPTIONS=-Dfile.encoding=UTF8
161161
download_tassl
162-
163162
LOG_INFO "------ download_binary: v3.0.0---------"
164163
download_build_chain "v3.0.0"
165164
download_binary "v3.0.0"
@@ -184,6 +183,13 @@ rm -rf ./bin
184183
LOG_INFO "------ download_build_chain: v3.3.0---------"
185184
download_binary "v3.3.0"
186185
download_build_chain "v3.3.0"
186+
LOG_INFO "------ check_standard_node---------"
187+
check_standard_node "true" "sm" "-s"
188+
rm -rf ./bin
189+
190+
LOG_INFO "------ download_build_chain: v3.4.0---------"
191+
download_binary "v3.4.0"
192+
download_build_chain "v3.4.0"
187193
LOG_INFO "------ check_wasm_node---------"
188194
check_wasm_node "false"
189195
LOG_INFO "------ check_standard_node---------"

.codecov.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ coverage:
77
status:
88
project:
99
default:
10-
target: "30%"
10+
target: "40%"
1111
threshold: "1%"
1212
if_not_found: success
1313
patch:

.github/workflows/workflow.yml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,16 +41,17 @@ jobs:
4141
if: runner.os == 'macOS'
4242
run: |
4343
brew install [email protected] ccache
44-
- name: Set up JDK 1.8
44+
- name: Set up JDK 1.8.0.345
4545
uses: actions/setup-java@v3
4646
with:
47-
distribution: 'zulu'
48-
java-version: '8.0.345'
47+
distribution: 'zulu'
48+
java-version: '8.0.345'
4949
- name: run build test
5050
if: runner.os == 'Windows'
5151
run: ./gradlew.bat build
5252
- name: run integration testing
53-
if: runner.os != 'Windows'
53+
# FIXME: macOS WASM integration testing failed
54+
if: runner.os != 'Windows' && runner.os != 'macOS'
5455
run: /bin/bash .ci/ci_check.sh
5556

5657
build-centos:
@@ -87,6 +88,11 @@ jobs:
8788
java-version: '8.0.345'
8889
- name: install CentOS dependencies
8990
run: yum install -y epel-release centos-release-scl wget which git openssl-devel openssl tree
91+
- name: Set up JDK 1.8.0.345
92+
uses: actions/setup-java@v3
93+
with:
94+
distribution: 'zulu'
95+
java-version: '8.0.345'
9096
- name: run build test
9197
run: /bin/bash gradlew build --info
9298
- name: run integration testing

build.gradle

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,26 +16,26 @@ ext {
1616
if (!project.hasProperty("ossrhPassword")) {
1717
ossrhPassword = "xxx"
1818
}
19-
jacksonVersion = '2.14.2'
19+
jacksonVersion = '2.14.3'
2020
commonsIOVersion = '2.11.0'
2121
commonsLang3Version = '3.12.0'
2222
toml4jVersion = "0.7.2"
2323
bcprovJDK15onVersion = '1.69'
2424
webankJavaCryptoVersion = "1.0.3"
2525
junitVersion = '4.13.2'
2626
commonsCollections4Version = "4.4"
27-
guavaVersion = '31.1-jre'
2827
bcosSdkJniVersion = "3.4.0"
2928
slf4jApiVerison = '1.7.36'
3029
mockitoVersion = '4.8.0'
3130
gsonVersion = '2.10.1'
31+
tarsSDKVersion = '3.5.0-SNAPSHOT'
3232
}
3333

3434
// check.dependsOn integrationTest
3535
// integrationTest.mustRunAfter test
3636
allprojects {
3737
group = 'org.fisco-bcos.java-sdk'
38-
version = '3.4.0'
38+
version = '3.5.0-SNAPSHOT'
3939

4040
apply plugin: 'maven-publish'
4141
apply plugin: 'idea'
@@ -60,15 +60,14 @@ allprojects {
6060
repositories {
6161
mavenCentral()
6262
maven { url "https://maven.aliyun.com/nexus/content/groups/public/" }
63-
maven { url "https://oss.sonatype.org/service/local/staging/deploy/maven2" }
6463
maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
64+
maven { url "https://oss.sonatype.org/service/local/staging/deploy/maven2" }
6565
}
6666

6767
dependencies {
6868
api("org.slf4j:slf4j-api:${slf4jApiVerison}")
6969
testImplementation("junit:junit:${junitVersion}")
7070
testImplementation("org.apache.commons:commons-collections4:${commonsCollections4Version}")
71-
testImplementation("com.google.guava:guava:${guavaVersion}")
7271
testImplementation("org.mockito:mockito-core:${mockitoVersion}")
7372
}
7473

@@ -126,6 +125,7 @@ googleJavaFormat {
126125
}
127126

128127
dependencies {
128+
api("org.fisco-bcos:fisco-bcos-tars-sdk" + ":${tarsSDKVersion}")
129129
api("org.fisco-bcos:bcos-sdk-jni:${bcosSdkJniVersion}") {
130130
exclude group : "org.slf4j"
131131
exclude group : "com.fasterxml.jackson.core"
@@ -136,14 +136,10 @@ dependencies {
136136
api("org.apache.commons:commons-lang3:${commonsLang3Version}")
137137
api("com.fasterxml.jackson.core:jackson-databind:${jacksonVersion}")
138138
api("commons-io:commons-io:${commonsIOVersion}")
139-
// api("com.webank:key-mini-toolkit:${keyMiniToolkit}")
140139
api("com.webank:webank-blockchain-java-crypto:${webankJavaCryptoVersion}")
141140
api("com.moandjiezana.toml:toml4j:${toml4jVersion}") {
142141
exclude group: "com.google.code.gson"
143142
}
144-
// api("org.apache.commons:commons-configuration2:${config2Version}"){
145-
// exclude group: "commons-logging"
146-
// }
147143

148144
integrationTestImplementation project
149145
integrationWasmTestImplementation project
@@ -183,6 +179,7 @@ javadoc {
183179
task sourcesJar(type: Jar) {
184180
from sourceSets.main.allJava
185181
archiveClassifier = 'sources'
182+
duplicatesStrategy = 'warn'
186183
}
187184

188185
task javadocJar(type: Jar) {
@@ -209,7 +206,6 @@ tasks.withType(Test) {
209206
publishing {
210207
publications {
211208
mavenJava(MavenPublication) {
212-
213209
artifactId "fisco-bcos-" + project.name
214210
groupId project.group
215211
version project.version
@@ -262,7 +258,7 @@ publishing {
262258

263259
jar {
264260
// destinationDir file('dist/apps')
265-
archiveName "fisco-bcos-" + project.name + '-' + project.version + '.jar'
261+
archiveFileName="fisco-bcos-" + project.name + '-' + project.version + '.jar'
266262
exclude '**/*.xml'
267263
exclude '**/*.properties'
268264

src/integration-test/java/org/fisco/bcos/sdk/v3/test/BcosSDKTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -337,9 +337,13 @@ public void onError(Response errorResponse) {
337337
BcosTransaction transaction1 = client.getTransaction(txHash, false);
338338
BcosTransactionReceipt transactionReceipt = client.getTransactionReceipt(txHash, false);
339339
if (client.getChainCompatibilityVersion().compareTo(EnumNodeVersion.BCOS_3_2_0.toVersionObj()) >= 0) {
340+
Assert.assertEquals(extraData, receipt.getExtraData());
340341
Assert.assertEquals(extraData, transaction1.getResult().getExtraData());
341342
Assert.assertEquals(extraData, transactionReceipt.getResult().getExtraData());
343+
Assert.assertFalse(receipt.getInput().isEmpty());
342344
}
345+
Assert.assertFalse(transactionReceipt.getResult().getInput().isEmpty());
346+
Assert.assertFalse(transaction1.getResult().getInput().isEmpty());
343347

344348
// get 2nd block
345349
block1 =

src/integration-test/java/org/fisco/bcos/sdk/v3/test/transaction/decoder/EventDecodeTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
*/
1515
package org.fisco.bcos.sdk.v3.test.transaction.decoder;
1616

17-
import com.google.common.collect.Lists;
17+
import java.util.ArrayList;
1818
import java.util.List;
1919
import java.util.Map;
2020
import org.fisco.bcos.sdk.v3.BcosSDK;
@@ -53,7 +53,7 @@ public void testDecode() throws Exception {
5353
client, client.getCryptoSuite().getCryptoKeyPair(), abiFile, binFile);
5454
ContractCodec contractCodec = new ContractCodec(client.getCryptoSuite(), client.isWASM());
5555
// deploy
56-
List<Object> params = Lists.newArrayList();
56+
List<Object> params = new ArrayList<>();
5757
params.add(1);
5858
params.add("test2");
5959
TransactionResponse response = manager.deployByContractLoader("ComplexSol", params);

src/integration-test/java/org/fisco/bcos/sdk/v3/test/transaction/decoder/TransactionDecoderServiceTest.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,9 @@
1414
*/
1515
package org.fisco.bcos.sdk.v3.test.transaction.decoder;
1616

17-
import com.google.common.collect.Lists;
1817
import java.math.BigInteger;
19-
import java.util.ArrayList;
20-
import java.util.List;
21-
import java.util.Map;
18+
import java.util.*;
19+
2220
import org.fisco.bcos.sdk.v3.BcosSDK;
2321
import org.fisco.bcos.sdk.v3.client.Client;
2422
import org.fisco.bcos.sdk.v3.model.ConstantConfig;
@@ -57,7 +55,7 @@ public void testDecode() throws Exception {
5755
TransactionProcessorFactory.createAssembleTransactionProcessor(
5856
client, client.getCryptoSuite().getCryptoKeyPair(), abiFile, binFile);
5957
// deploy
60-
List<Object> params = Lists.newArrayList();
58+
List<Object> params = new ArrayList<>();
6159
params.add(1);
6260
params.add("test2");
6361
TransactionResponse response = manager.deployByContractLoader(contractName, params);
@@ -71,9 +69,10 @@ public void testDecode() throws Exception {
7169
contractName,
7270
contractAddress,
7371
"incrementUint256",
74-
Lists.newArrayList(BigInteger.valueOf(1)));
72+
Collections.singletonList(BigInteger.ONE));
7573
TransactionResponse transactionResponseWithoutValues =
7674
decoder.decodeReceiptWithoutValues(abi, transactionReceipt);
75+
Assert.assertEquals(0, transactionResponseWithoutValues.getReturnCode());
7776
TransactionResponse transactionResponseWithValues =
7877
decoder.decodeReceiptWithValues(abi, "incrementUint256", transactionReceipt);
7978
Assert.assertEquals("Success", transactionResponseWithValues.getReceiptMessages());
@@ -83,8 +82,8 @@ public void testDecode() throws Exception {
8382
}
8483
// setBytesMapping
8584
{
86-
List<Object> s = Lists.newArrayList("2".getBytes());
87-
List<Object> paramsSetBytes = new ArrayList<Object>();
85+
List<Object> s = Collections.singletonList("2".getBytes());
86+
List<Object> paramsSetBytes = new ArrayList<>();
8887
paramsSetBytes.add(s);
8988
TransactionReceipt transactionReceipt2 =
9089
manager.sendTransactionAndGetReceiptByContractLoader(
@@ -96,7 +95,7 @@ public void testDecode() throws Exception {
9695
Assert.assertEquals(
9796
transactionResponse2.getReceiptMessages(), "Bytes array is less than 2");
9897

99-
List<Object> s2 = Lists.newArrayList("2".getBytes(), "3".getBytes());
98+
List<Object> s2 = Arrays.asList("2".getBytes(), "3".getBytes());
10099
List<Object> paramsSetBytes2 = new ArrayList<>();
101100
paramsSetBytes2.add(s2);
102101
TransactionReceipt transactionReceipt =

0 commit comments

Comments
 (0)