Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix junit reference #6056

Closed
wants to merge 114 commits into from
Closed

Fix junit reference #6056

wants to merge 114 commits into from

Conversation

TheLimeGlass
Copy link
Collaborator

Description

  • I added a package com.skriptlang.skript.test.registration which contains syntaxes that are only registered for the test system, I don't believe they should be in the Skript main package unless they're designed for the test system.
  • Fixed an issue where the JUnit system would not clear the previous test class from the TestTracker when it requires a completed task.
  • Allowed JUnit classes to register syntaxes after initialization.
  • Removed SkriptLogger.setVerbosity(Verbosity.DEBUG); from Skript.class as there is a parameter for it now from gradle task.

Fixed originally in #5219

Target Minecraft Versions: none
Requirements: none
Related Issues: none

TheLimeGlass and others added 30 commits November 23, 2022 23:39
@TheLimeGlass
Adds a SetEffect lang util class
05dd094
@TheLimeGlass @APickledWalrus
Update src/main/java/ch/njol/skript/effects/base/package-info.java
c61de4c 
Co-authored-by: Patrick Miller <apickledwalrus@gmail.com>
@TheLimeGlass
Merge branch 'master' into feature/seteffect
27842f9
@TheLimeGlass
Update set effect and apply changes
04ecbb5
@TheLimeGlass
Some JUnit changes
4f7927b
@TheLimeGlass
Some JUnit changes
38f86f2
@TheLimeGlass
Update JavaDoc
17eb299
@TheLimeGlass
Remove debug messages
403fddb
@Moderocky
Merge branch 'master' into feature/seteffect
c865458
@TheLimeGlass
Adds a negated method for developers to check against
992ccf0
@TheLimeGlass
Merge branch 'master' into feature/seteffect
40f1fad
@TheLimeGlass
Aplpy changes
622727c
@TheLimeGlass
Merge branch 'feature/seteffect' of https://github.com/SkriptLang/Skript
deb7306 
 into feature/seteffect
@TheLimeGlass
Update SetEffect.java
445ba9b
@TheLimeGlass
Add unformal 'not' optional pattern
4cd3b51
@TheLimeGlass
Merge branch 'master' into feature/seteffect
e5a5881
@TheLimeGlass
Update SetEffect.java
873a4f1
@TheLimeGlass
Update SetEffect.java
56b4af2
@TheLimeGlass
Add is climbing (#5618)
aeaee7a
@UnderscoreTud @TheLimeGlass
Add Vector Projection Expression (#5482)
b91b3a8 
add vector projection expression

Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
@NotSoDelayed
Condition LivingEntity is Jumping (#5359)
97caf93
@dependabot @TheLimeGlass
Bump io.papermc.paper:paper-api from 1.19.4-R0.1-SNAPSHOT to 1.20-R0.…
2c427fa 
…1-SNAPSHOT (#5739)

* Bump io.papermc.paper:paper-api

Bumps io.papermc.paper:paper-api from 1.19.4-R0.1-SNAPSHOT to 1.20-R0.1-SNAPSHOT.

---
updated-dependencies:
- dependency-name: io.papermc.paper:paper-api
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update and rename paper-1.19.3.json to paper-1.20.json

* Update build.gradle

* Update gradle.properties

* Update default.lang

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
@UnderscoreTud
Add InventoryDragEvent (#5474)
3dd5394
@sovdeeth
Adds EffToggleCanPickUpItems + CondCanPickUpItems (#5357)
a5f0f4f
@AyhamAl-Ali
🚀 Add anvil repair cost expression (#4617)
1949a23
@sovdeeth
Adds Clamp Function (#5573)
302f8f9
@dependabot @TheLimeGlass
Bump io.papermc.paper:paper-api from 1.20-R0.1-SNAPSHOT to 1.20.1-R0.…
2e5dc2e 
…1-SNAPSHOT (#5746)

* Bump io.papermc.paper:paper-api

Bumps io.papermc.paper:paper-api from 1.20-R0.1-SNAPSHOT to 1.20.1-R0.1-SNAPSHOT.

---
updated-dependencies:
- dependency-name: io.papermc.paper:paper-api
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update build.gradle

* Update gradle.properties

* Update and rename paper-1.20.json to paper-1.20.1.json

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
@TheLimeGlass
Add quit reason (#5763)
ce1888c 
* Add quit reason

* Cleanup

* Versioning
@AyhamAl-Ali
🚀 Add loop-counter & Improve loops (#4595)
9d06f02
@sovdeeth
Add CondIsLeftHanded and EffHandedness (#5494)
e08ec47
ShaneBeee and others added 24 commits September 11, 2023 11:19
@ShaneBeee
EffPotion - fix potion duration issue (#5992)
0ea98d4
@MihirKohli
Rename Slot classinfo name to Slot (#6001)
b722cc3
@AyhamAl-Ali
🛠 Fix keep inv/exp and other bugs related to death event (#5658)
869a5de
@TheLimeGlass
Fix event-item not being present in book sign (#5877)
bd134d0
@Moderocky
Merge branch 'dev/patch' into dev/feature
f70d005
@AyhamAl-Ali
⚒️ Add NoDoc annotation to test elements (#6006)
cd731f1
@dependabot @TheLimeGlass @Moderocky
Bump org.easymock:easymock from 5.1.0 to 5.2.0 (#5970)
27e2077 
Bumps [org.easymock:easymock](https://github.com/easymock/easymock) from 5.1.0 to 5.2.0.
- [Release notes](https://github.com/easymock/easymock/releases)
- [Changelog](https://github.com/easymock/easymock/blob/master/ReleaseNotes.md)
- [Commits](easymock/easymock@easymock-5.1.0...easymock-5.2.0)

---
updated-dependencies:
- dependency-name: org.easymock:easymock
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
Co-authored-by: Moderocky <admin@moderocky.com>
@TheLimeGlass
Add GitHub's gradle wrapper checker for security (#6002)
07cc9d6 
Add GitHub's new gradle wrapper checker for security
@sovdeeth @Moderocky @AyhamAl-Ali
Fix NPE with invalid attributes and clean up ExprEntityAttribute (#5978)
5b59105 
* Avoid NPE and clean up class

* Update ExprEntityAttribute.java

* Update src/main/java/ch/njol/skript/expressions/ExprEntityAttribute.java

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>

* Update src/main/java/ch/njol/skript/expressions/ExprEntityAttribute.java

---------

Co-authored-by: Moderocky <admin@moderocky.com>
Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
@TheLimeGlass @AyhamAl-Ali @Moderocky
Fix cast throwing if existing variable for command storage exists (#5942
66b08b8 
)

* Fix cast throwing if existing variable for command storage exists

* Update src/main/java/ch/njol/skript/command/ScriptCommand.java

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>

---------

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
Co-authored-by: Moderocky <admin@moderocky.com>
@TheLimeGlass @Moderocky
Allow and with for the title effect (#6010)
3dfc252 
* Allow and with for the title effect

* apply changes

---------

Co-authored-by: Moderocky <admin@moderocky.com>
@TheLimeGlass @AyhamAl-Ali @Moderocky
Add get all armour of living entities (#5456)
4b5579c 
* Add get all armour

* Apply changes

* Update src/main/java/ch/njol/skript/expressions/ExprArmorSlot.java

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>

---------

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
Co-authored-by: Moderocky <admin@moderocky.com>
@NotSoDelayed @TheLimeGlass @Moderocky
Enhance Shearing Related Elements (#5571)
38bfa0a 
* Enhance

* Enhance (finish)

* Enhance (finish x2)

* Docs

* Eclipse -> Jetbrains annotation

* Requested change

* Requested change

* Requested change

Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>

* Camel case static final field

* Requested change and minor correction on comment

* Remove un-needed null warning suppression

* Requested change + fix docs

---------

Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
Co-authored-by: Moderocky <admin@moderocky.com>
@TheLimeGlass @AyhamAl-Ali @Moderocky
Move documentation into its own folder (#5580)
d55e5a6 
* Move documentation into it's own folder

* Update src/main/java/ch/njol/skript/doc/Documentation.java

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>

* Update src/main/java/ch/njol/skript/doc/Documentation.java

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>

---------

Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
Co-authored-by: Moderocky <admin@moderocky.com>
@UnderscoreTud @Moderocky
Allow 'continue' the continue outer loops (#6024)
41349d1 
* Allow continuing outer loops

* Requested Changes

* Add tests

---------

Co-authored-by: Moderocky <admin@moderocky.com>
@kiip1
Yggdrasil Cleanup (#5408)
fa43514
@dependabot
Bump actions/checkout from 3 to 4 (#6029)
293e824 
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...
@dependabot @AyhamAl-Ali @TheLimeGlass
Bump org.easymock:easymock from 5.1.0 to 5.2.0 (#6030)
5bedcee 
Bumps [org.easymock:easymock](https://github.com/easymock/easymock) from 5.1.0 to 5.2.0.
- [Release notes](https://github.com/easymock/easymock/releases)
- [Changelog](https://github.com/easymock/easymock/blob/master/ReleaseNotes.md)
- [Commits](easymock/easymock@easymock-5.1.0...easymock-5.2.0)

---
updated-dependencies:
- dependency-name: org.easymock:easymock
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
@dependabot @AyhamAl-Ali @TheLimeGlass
Bump org.gradle.toolchains.foojay-resolver-convention from 0.5.0 to 0…
b7c4cdd 
….7.0 (#6031)

Bump org.gradle.toolchains.foojay-resolver-convention

Bumps org.gradle.toolchains.foojay-resolver-convention from 0.5.0 to 0.7.0.

---
updated-dependencies:
- dependency-name: org.gradle.toolchains.foojay-resolver-convention
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Ayham Al Ali <20037329+AyhamAl-Ali@users.noreply.github.com>
Co-authored-by: LimeGlass <16087552+TheLimeGlass@users.noreply.github.com>
@TheLimeGlass
Update gradle to gradle 8.3 (#5974)
1ffe86f
@TheLimeGlass
Merge branch 'dev/feature' into feature/seteffect
17bd6a3
@TheLimeGlass
Use void method for applying
28d3816
@TheLimeGlass
Apply changes to the tests
792364a
@TheLimeGlass
Fix JUnit not clearing references from the last JUnit test
1245fc4
@TheLimeGlass TheLimeGlass added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Sep 22, 2023
@TheLimeGlass TheLimeGlass changed the base branch from dev/feature to dev/patch September 22, 2023 05:28
@TheLimeGlass TheLimeGlass deleted the fix/junit-reference branch September 22, 2023 05:29
@TheLimeGlass TheLimeGlass restored the fix/junit-reference branch September 22, 2023 05:29
@TheLimeGlass TheLimeGlass deleted the fix/junit-reference branch September 22, 2023 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.