Paths

Table of Contentst

Differential D3244

Update mockito to latest version
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F10353712: D3244.id8336.diff
	Tue, Jul 1, 19:22

	F10353686: D3244.id8335.diff
	Tue, Jul 1, 19:09

	F10353580: D3244.id.diff
	Tue, Jul 1, 17:56

	F10345145: D3244.id8334.diff
	Tue, Jul 1, 05:11

	Unknown Object (File)
	Mon, Jun 30, 21:27

	Unknown Object (File)
	Thu, Jun 26, 13:28

	Unknown Object (File)
	Mon, Jun 23, 23:58

	Unknown Object (File)
	Mon, Jun 23, 23:13

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.Sep 10 2023, 18:21

EshaanAgg created this revision.

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

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

Issue: T1876

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

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

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

dereckson added inline comments.Sep 10 2023, 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.Sep 11 2023, 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.Sep 21 2023, 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.Sep 21 2023, 16:10

Closed by commit rDS19378908c94e: Update mockito to latest version (authored by EshaanAgg, committed by dereckson). · Explain WhySep 21 2023, 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)Sep 21 2023, 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...