🗑️ Deprecate RemovePlugin constructor without plugin name

✅ Add test for RemovePlugin

✅ Add test for RemovePersistentRequest

✅ Add test for RemovePeer

🗑️ Deprecate ReloadPlugin constructor without plugin name

✅ Add test for ReloadPlugin

✅ Add test for ModifyPersistentRequest

🗑️ Deprecate ModifyConfig without identifier

✅ Add test for ModifyConfig

🗑️ Deprecate official-source functionality in LoadPlugin

✅ Add tests for LoadPlugin

🗑️ Deprecate ListPersistentRequests constructor without identifier

🎨 Add ListPersistentRequests constructor with identifier

✅ Add test for ListPersistentRequests

🗑️ Deprecate ListPeers constructor with flags

✅ Add test for ListPeers

🗑️ Deprecate ListPeerNotes constructor without both identifiers

✅ Add test for ListPeerNotes

💚 Archive javadoc in binary stage

🗑️ Deprecate ListPeer constructor without both identifiers

✅ Add test for ListPeer

♻️ Move set-flag-and-verify method to helper class

✅ Add test for GetRequestStatus

🗑️ Deprecate GetPluginInfo constructor without plugin name

✅ Add test for GetPluginInfo

💚 Remove invocation of Jenkins’ jacoco plugin in CI build

✏️ Fix typo in parameter name

🗑️ Deprecate GetConfig constructor without identifier

✅ Add test for GetConfig command

♻️ Use non-deprecated constructor

♻️ Rename constructor parameter in GenerateSSK

🗑️ Deprecate constructor without identifier

✅ Add test for GenerateSSK message

✅ Add tests for file entries

🔥 Remove Guava dependency

✅ Add tests for FCPPluginReply

✅ Add tests for FCPPluginMessage

✅ Add tests for FcpMessage

✅ Add test for FcpListenerManager

🧪 Add test for modifyConfig()

✅ Add tests for getConfig()

✅ Add tests for getNodeInformation()

💚 Use verify for recording JaCoCo coverage correctly

✅ Add tests for sendPluginMessage()

💚 Use test phase instead of verify phase

✅ Add test for SimpleProgress being used in getRequests()

🐛 Fix sending of requests

The terminating message came too early, fucking up everything.

✅ Add test for GetFailed being used in getRequests()

✅ Add test for DataFound for unknown identifiers not touching anything else

✅ Add test for DataFound being used in getRequests()

♻️ Rename flag for including global requests

✅ Add test for getting global requests

✅ Add test for getPutRequests() and refactor method

♻️ Add method for easier creation of single-message responses

✅ Add test for getGetRequests() and refactor method

🚧 Add matchers for high-level requests

✅ Add tests for modifyPeerNote()

✏️ Fix typo in Java 17 build

✅ Add test for getPeerNote()

🏗️ Add Jenkinsfile

🐛 Make removePeer() return normally when node identifier is unknown

Also, add a test!

✅ Make tests more realistic by running listeners in a different thread

✅ Deprecate and replace modifyPeer(), add test

🐛 Only accept matchers for Strings on FCP message field values

✅ Add tests for ModifyPeer, deprecate and replace a method

♻️ Use Stream API to filter returned peers

🔥 Remove fixed FIXMEs

Fuck the person who added FIXMEs in the source code and didn’t bother to
actually fix the issues.

It’s me. I was the one who did that.

✅ Add tests for addPeer() methods

And add Trust and Visibility parameters to all addPeer() methods.

✅ Add tests for getSeedPeers()

♻️ Move Peer creation into its own class

✅ Add test for Peer

And in the process also fix a bug!

🚧 Add matchers for ARKs, Versions, and DSAGroups

✅ Add test for getOpennetPeers()

✅ Add test for getDarknetPeers()

✨ Add tests for getPeers() method

Also, some more matchers for FCP messages as well as Peer objects were
added.

🐛 Only evaluate the message function once

♻️ Add doNothing() method that can be used to consume a message without doing anything

♻️ Extract InputStream matcher to its own file

✅ Add beginning of test for FcpClient

👷 Fix URL of Maven repository

📝 Slightly improve javadoc

✅ Prevent test from starting if local server is not yet up

📝 Add some @param tags to appease javadoc

✅ Add more tests for DefaultFcpConnection

✨ Add String-based dispatcher implementation of FcpListener

🚸 Add more methods to PersistentPutDir

Also, add a test.

✅ Add first couple of tests for FCP connection

🔀 Merge branch 'refactoring/increase-testability' into next

I’m not sure why I have not made all these changes on next, next is the
perfect branch for this. :)

✨ Add additional getters for PersistentPut fields

✅ Add test for Freenet-specific Base64 implementation

📝 Fix typo in documentation

🔥 Remove unnecessary configuration

⬆️ Update JUnit and Maven plugins

The JUnit update fixes a problem with coverage recording, see
https://github.com/jacoco/jacoco/issues/1310#issuecomment-1118191086 and
https://github.com/junit-team/junit4/issues/1652.

🔀 Merge remote-tracking branch 'github/refactoring/increase-testability' into refactoring/increase-testability

♻️ Refactor the FileEntry classes

And by that I mean “remove most of them.” The generalised FileEntry that
exposes a Map and an InputStream is absolutely adequate and removes the
instanceof check in ClientPutComplexDir.

♻️ Slightly refactor the file entries

More to come!

✅ Add test for FcpUtils

Also, fix a bug with the long-parsing.

♻️ Move TempInputStream to its own class

✅ Add test for “ConfigData”

✅ Add test for “BaseMessage”