ljacqu
6422f90114
Create Wrapper for instances / revise MockUtils
...
- Add test resources folder
- Create basic test for Messages (todo: add concrete tests)
- Create WrapperMock
- Change UtilsTest (todo: make it work)
2015-11-23 23:25:03 +01:00
DNx5
d7f9b602c7
update some Utils method
2015-11-24 04:43:29 +07:00
Tim Visée
09067ddbd1
Removed redundant code, fixed some warnings, other minor fixes
2015-11-23 22:18:04 +01:00
Tim Visée
82bf0f45ca
Finalized some fields, removed redundant code, various other fixes
2015-11-23 22:14:03 +01:00
Tim Visée
83e5d726af
Improved various code bits
2015-11-23 22:12:01 +01:00
Tim Visée
1dd540b9c0
Fixed some redundant casts and definitions
2015-11-23 22:09:15 +01:00
Tim Visée
47b1d00ccf
Fixed various spelling and grammar issues
2015-11-23 22:07:32 +01:00
Tim Visée
c3fa8b272f
Added personal dictionary
2015-11-23 21:55:36 +01:00
Tim Visée
3571bcc337
Include everyone's project dictionary
2015-11-23 21:55:25 +01:00
Tim Visée
36c50c4507
Reformatted code with new code style
2015-11-23 21:46:34 +01:00
Tim Visée
b2db25d16c
Improved JavaDocs code style, some minor improvements
2015-11-23 21:45:23 +01:00
Tim Visée
f15eaae67f
Some minor automatic fixes and changes
2015-11-23 21:43:40 +01:00
Tim Visée
1cdf49f7e1
Disabled wrap when typing option
2015-11-23 21:40:34 +01:00
Tim Visée
53117928f5
Reformatted all files once again
2015-11-23 21:40:11 +01:00
Tim Visée
4a2a5ed309
Improved .gitignore
2015-11-23 21:39:42 +01:00
Tim Visée
4f8e1c6e91
Set the default line-separator to LF, for Linux/Unix style
2015-11-23 21:36:41 +01:00
Gabriele C
ab77b1291d
codestyle optimizations
2015-11-23 21:33:42 +01:00
Tim Visée
2e868c7492
Reformatted all code files, cleaned up the project
2015-11-23 21:33:08 +01:00
Gabriele C
cffcafd36b
codestyle fix
2015-11-23 21:29:09 +01:00
Tim Visée
aa6d24862d
Reverted extreme project code style, using default now
2015-11-23 21:25:57 +01:00
Tim Visée
6519f1d865
Set up an extreme code style test
2015-11-23 21:13:15 +01:00
ljacqu
8719cce334
Create enum for message keys in Messages
2015-11-23 21:13:12 +01:00
Tim Visée
b0cc6826b4
Include the project's code style, fixed invalid .idea directory being ignored
2015-11-23 21:06:36 +01:00
Tim Visée
a9900336ae
Don't ignore the .gitignore file itself
2015-11-23 21:06:35 +01:00
Tim Visée
87566cb8ea
Updated .gitignore to include project's code style settings file
2015-11-23 21:06:35 +01:00
Gabriele C
10435657c5
Fix enableBackup option
2015-11-23 20:59:25 +01:00
Gabriele C
614aa9d5e5
Fix noTeleport in the PlayerListener
2015-11-23 20:59:13 +01:00
Tim Visée
cf475932fe
Added basic .editorconfig configuration
2015-11-23 20:43:25 +01:00
AuthMe-Team
9ec2d6d059
Stuff from the common floobits workspace
...
Author: AuthMe-Team <AuthMeTeam@123NoEmail.com>
2015-11-23 20:23:52 +01:00
Gabriele C
69a09aec17
cleanup
2015-11-22 21:06:08 +01:00
Tim Visée
da662afcf4
Created method to get an exception stack trace as a string
2015-11-22 21:01:31 +01:00
ljacqu
215fedc585
Minor refactoring in CustomConfiguration / Messages
...
- CustomConfiguration should be abstract
- Reduce visibility of internal method
- Fix typo in English messages
- Remove or replace unnecessary javadoc
2015-11-22 02:02:49 +01:00
ljacqu
e456203fc6
Refactor Utils#getOnlinePlayers and add background info
2015-11-22 01:30:07 +01:00
ljacqu
fc3f685de0
Create UtilsTest
...
Had to create a getGameServer() method in AuthMe in order to be able to catch this call in tests and replace it with a mock implementation of the actual server. The supertype method getServer() is declared as final, which makes it impossible to mock with Mockito.
While there are frameworks that manipulate the bytecode to mock final calls, it is much easier and less risky to simply delegate the server retrieval through another method.
2015-11-21 23:09:04 +01:00
Tim Visée
1ba18439e9
Fixed invalid method arguments
2015-11-21 21:08:07 +01:00
Tim Visée
ada991904b
Improved purgePermissions method
2015-11-21 21:07:32 +01:00
Tim Visée
e45aa88504
Fixed result not being returned
2015-11-21 21:03:50 +01:00
Tim Visée
18af8f3171
Fixed addLimboPlayer method
2015-11-21 21:02:53 +01:00
Tim Visée
0a5c081732
Added method to permissions manager to get primary group of player
2015-11-21 20:59:23 +01:00
Tim Visée
89b5444d69
Fixed addNormal method, updated it's JavaDocs
2015-11-21 20:50:12 +01:00
Tim Visée
7902cd87bb
Added return statement for setGroup method, updated it's JavaDocs
2015-11-21 20:47:56 +01:00
Gabriele C
579c51f4a2
Remove the old Vault group management system. (THIS COMMIT BREAKS THE COMPILATION)
2015-11-21 20:16:07 +01:00
Tim Visée
579b7e7b97
Improved some grammar and variable names
2015-11-21 19:24:25 +01:00
Tim Visée
a176aba350
Fixed horrible typo, implemented dynamic plugin name
2015-11-21 19:22:23 +01:00
Tim Visée
30b4cfe44e
Added notice to legacy permissions instance in main class
2015-11-21 19:18:56 +01:00
ljacqu
b3d0a71dec
Merge ListUtil into StringUtil; refactor HelpSyntaxHelper + create test
...
The HelpSyntaxHelper had suppressed warnings for string concatenation within StringBuilder - the point of the StringBuilder is that it is faster when you use it to concatenate many elements. If you still use string concatenation with + within these calls it beats the purpose.
2015-11-21 17:19:11 +01:00
ljacqu
a3f24bcb9a
Create test for HelpSyntaxHelperTest
2015-11-21 17:18:58 +01:00
ljacqu
58dc15123c
Add tests for LogoutCommand and RegisterCommand. Add more generic mockUtil
2015-11-21 17:18:45 +01:00
ljacqu
4e8614fdf7
Add test for LoginCommand; create AuthMe mock test util
...
Had to create a getter for the Management instance in the AuthMe class for mocking, but fields should generally not be accessed globally. Hopefully soon we will be able to make the field private.
2015-11-21 17:18:28 +01:00
ljacqu
d81ef3168e
Command refactor - remove unused fields, reduce variable "scope"
...
Minor refactorings in the command section for familiarization.
1. Removed suppressWarning("Deprecated") - the method is deprecated for a reason and we should be made aware of that.
2. Removed same javadoc on ExecutableCommand implementation that just had the same as the interface (this is just clutter; @Override signals that it's an implementing class and a developer can view the superclass javadoc)
3. In places where the AuthMe instance was retrieved at the top but used at the very bottom, moved it to the bottom to reduce its "scope"
2015-11-21 17:18:14 +01:00