Paths

Table of Contentst

Differential D3244

Update mockito to latest version
ClosedPublic
Actions

Authored by EshaanAgg on Sun, Sep 10, 18:21.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 24, 16:47

	Unknown Object (File)
	Sun, Sep 24, 09:43

	Unknown Object (File)
	Sat, Sep 23, 21:55

	Unknown Object (File)
	Sat, Sep 23, 18:51

	Unknown Object (File)
	Sat, Sep 23, 10:48

	Unknown Object (File)
	Fri, Sep 22, 23:43

	F2299622: D2344.log
	Wed, Sep 20, 22:31

	Unknown Object (File)
	Mon, Sep 18, 14:11

Subscribers

None

Details

Reviewers

Maniphest Tasks

T1876: Upgrade mockito code to 1.x

Commits

rDS19378908c94e: Update mockito to latest version

Diff Detail

Repository

rDS Nasqueron Datasources

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

EshaanAgg requested review of this revision.Sun, Sep 10, 18:21

EshaanAgg created this revision.

Harbormaster completed remote builds in B5116: Diff 8334.Sun, Sep 10, 18:21

dereckson added a task: T1876: Upgrade mockito code to 1.x.Sun, Sep 10, 18:28

Issue: T1876

Harbormaster completed remote builds in B5117: Diff 8335.Sun, Sep 10, 18:28

EshaanAgg mentioned this in T1876: Upgrade mockito code to 1.x.Sun, Sep 10, 18:29

dereckson added a reviewer: dereckson.Sun, Sep 10, 19:12

dereckson added inline comments.Sun, Sep 10, 19:19

opendatasoft-explore-api/Cargo.toml
26–27	Perhaps we can now remove this line.
opendatasoft-explore-api/tests/requests_test.rs
58	If we use the full variable name we'll avoid a question "what's that `s` lost at the end of the method?"

Thanks for the upgrade. Will have a look.

We can also prepare a CI job on Jenkins now we've tests to run.

upd: Incorporate changes mentioned in review

Harbormaster completed remote builds in B5118: Diff 8336.Mon, Sep 11, 06:30

@dereckson Added the suggested changes. We can work on adding a Jenkin's pipeline in a new issue to keep the issues isolated and to the point. Does that work?

D2344.log2 KBDownload

All tests pass successfully o/

dereckson accepted this revision.Thu, Sep 21, 16:10

dereckson retitled this revision from chore: Update mockito to latest version to Update mockito to latest version.

This revision is now accepted and ready to land.Thu, Sep 21, 16:10

Closed by commit rDS19378908c94e: Update mockito to latest version (authored by EshaanAgg, committed by dereckson). · Explain WhyThu, Sep 21, 16:11

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rDS19378908c94e: Update mockito to latest version.

dereckson attached a referenced file: F2299622: D2344.log. (Show Details)Thu, Sep 21, 16:11

Revision Contents
Changeset List

Path

Size

opendatasoft-explore-api/

3 lines

tests/

requests_test.rs

142 lines

Diff 8337

opendatasoft-explore-api/Cargo.toml

Loading...

opendatasoft-explore-api/tests/requests_test.rs

Loading...