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

Various bugfixes and code improvements #137

Merged
merged 46 commits into from
Apr 28, 2024
Merged

Various bugfixes and code improvements #137

merged 46 commits into from
Apr 28, 2024

Conversation

LaravelFreelancerNL
Copy link
Owner

Additions

  • Added DatabaseMigrations trait
  • Added DatabaseTruncation trait
  • Added getTables method to the schema builder
  • Added support for automatic key generator setting for autoincrement and uuid on id/_key fields
  • Added Self contained EDGE_COLLECTION constant to create-edge migration stub

Bugfixes

  • Fixed premature json encoding/decoding of casts.
  • Removed duplicate getDateFormat function
  • Fixed RefreshData not seeding the DB on the initial migrate:fresh
  • Removed nonsensical hasColumn method from blueprint. (commands are created first, then executed sequentially, a conditional command won't work in this scenario)
  • Fixed isBind regex in query grammar

chores

  • cleaned up test setup
  • improved general code quality
  • added tests

LaravelFreelancerNL and others added 30 commits February 28, 2024 17:42
@LaravelFreelancerNL @github-actions
Fix styling
d848d5c
@LaravelFreelancerNL
Fixed Schema::hasColumn & Schema::hasColumns
8863cd2
@LaravelFreelancerNL
Moved const to builder
74e8720
@LaravelFreelancerNL
Added orchestra/testbench migration path.
d59e6e8
@LaravelFreelancerNL
Fixed transactionCollections setting
e2080a6
@LaravelFreelancerNL
Self contained EDGE_COLLECTION constant
dba95c4
@LaravelFreelancerNL
Removed nonsensical hasColumn method from blueprint.
9353dff 
(commands are created first, then executed sequentially, a conditional command won't work in this scenario)
@LaravelFreelancerNL
Set all transaction collections first
c8d293e
@LaravelFreelancerNL
Removed unused imports
0268aee
@LaravelFreelancerNL
Style fixes
360f4c0
@LaravelFreelancerNL
Added support for automatic key generator setting for autoincrement a…
9637ddd 
…nd uuid on id/_key fields.

Added support for autoincrement offset (from)

Restructured schema tests

Added schema tests
@LaravelFreelancerNL @github-actions
Fix styling
e698742
@LaravelFreelancerNL
CQ improvement
b378cb6 
Restructured schema tests

Added schema tests
@LaravelFreelancerNL
Added tests to ensure whereAll/whereAny/orWhereAll/orWhereAny functio…
a800ad4 
…nality.
@LaravelFreelancerNL
Merge remote-tracking branch 'origin/chore/add-feature-tests' into ch…
0475581 
…ore/add-feature-tests
@LaravelFreelancerNL @github-actions
Fix styling
fe71f2e
@LaravelFreelancerNL
Fixed Schema::hasColumn & Schema::hasColumns
93b345a
@LaravelFreelancerNL
Moved const to builder
2838b5b
@LaravelFreelancerNL
Added orchestra/testbench migration path.
3728b2d
@LaravelFreelancerNL
Self contained EDGE_COLLECTION constant
d4d5af9
@LaravelFreelancerNL
Removed nonsensical hasColumn method from blueprint.
bc79c50 
(commands are created first, then executed sequentially, a conditional command won't work in this scenario)
@LaravelFreelancerNL
Removed unused imports
6e5eeb9
@LaravelFreelancerNL
Style fixes
1b9efb4
@LaravelFreelancerNL
Added support for automatic key generator setting for autoincrement a…
39aecb4 
…nd uuid on id/_key fields.

Added support for autoincrement offset (from)

Restructured schema tests

Added schema tests
@LaravelFreelancerNL
CQ improvement
9f4969a 
Restructured schema tests

Added schema tests
@LaravelFreelancerNL
Added tests to ensure whereAll/whereAny/orWhereAll/orWhereAny functio…
585f90a 
…nality.
@LaravelFreelancerNL
WIP
e526c05
@LaravelFreelancerNL
Place test setup outside of tests directory.
ba105dc
@LaravelFreelancerNL
Fixed prompt answer
d10007f
@LaravelFreelancerNL
Fixed test setup
a7ee2e6
LaravelFreelancerNL and others added 16 commits April 28, 2024 19:59
@LaravelFreelancerNL
Using package own RefreshData trait to test it.
001feae 
Added ability to set realpath (deprecated in Laravel 11, but used for now) and migrations paths to be used by RefreshDatabase's migrate:fresh
@LaravelFreelancerNL
Removed todo for currently unsupported feature
f1c7f28
@LaravelFreelancerNL
Logfake dep supports Laravel 11. Removed the skip.
d31559f
@LaravelFreelancerNL
Removed duplicate getDateFormat function
615f437
@LaravelFreelancerNL
Cleanup
098684a
@LaravelFreelancerNL
Added DatabaseMigrations trait
62949ce
@LaravelFreelancerNL
Added DatabaseTruncation trait
ce53ddc
@LaravelFreelancerNL
Fixed premature json encoding/decoding of casts.
7d8e17d
@LaravelFreelancerNL
Added getTables
8063dd1
@LaravelFreelancerNL
Added tests and CQ improvements
3c37740
@LaravelFreelancerNL
Merge remote-tracking branch 'origin/chore/add-feature-tests' into ch…
1767e56 
…ore/add-feature-tests

# Conflicts:
#	TestSetup/Database/Migrations/2019_11_15_000000_create_children_edge_table.php
#	bin/qa.sh
#	composer.lock
#	src/Schema/Blueprint.php
#	src/Schema/Grammar.php
#	tests/Query/WheresTest.php
#	tests/Schema/AnalyzerTest.php
#	tests/Schema/SchemaBuilderTest.php
#	tests/Schema/TableTest.php
#	tests/TestCase.php
@LaravelFreelancerNL @github-actions
Fix styling
36c2df0
@LaravelFreelancerNL
Let CI use test scripts
c870cf8
@LaravelFreelancerNL
Merge remote-tracking branch 'origin/chore/add-feature-tests' into ch…
e56562f 
…ore/add-feature-tests
@LaravelFreelancerNL
Let CI use test scripts
c975213
@LaravelFreelancerNL
Convert testbench migrations
55b0928
@LaravelFreelancerNL LaravelFreelancerNL linked an issue Apr 28, 2024 that may be closed by this pull request
Confirm whereAll/whereAny functionality #122
Closed
@LaravelFreelancerNL LaravelFreelancerNL merged commit d20d04d into next Apr 28, 2024
12 of 13 checks passed
@LaravelFreelancerNL LaravelFreelancerNL deleted the chore/add-feature-tests branch April 28, 2024 18:36
@LaravelFreelancerNL LaravelFreelancerNL linked an issue Apr 28, 2024 that may be closed by this pull request
Ensure Eloquent compatibility with tests #92
Open
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Ensure Eloquent compatibility with tests Confirm whereAll/whereAny functionality
1 participant
@LaravelFreelancerNL