Skip to content

Add test for ColumnInterface#304

Merged
vjik merged 1 commit into
yiisoft:masterfrom
Tigrov:add_test_for_column_interface
Jul 28, 2023
Merged

Add test for ColumnInterface#304
vjik merged 1 commit into
yiisoft:masterfrom
Tigrov:add_test_for_column_interface

Conversation

@Tigrov

@Tigrov Tigrov commented Jul 28, 2023

Copy link
Copy Markdown
Member
Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues -

@what-the-diff

what-the-diff Bot commented Jul 28, 2023

Copy link
Copy Markdown

PR Summary

  • Modification of Test File
    The QueryBuilderTest.php is updated. This is a part of our testing suite that assures the quality and reliability of our code.

  • New Test Case Addition
    A new test case has been added for the alterColumn method. This method changes the characteristics of a column in our database.

  • Testing the SQL Statement Output
    This new test case ensures that when we modify a column and add a new unique constraint, the right SQL (Structured Query Language) statement is produced. SQL is the language we use to manage and manipulate our database.

  • Validation of Rules
    Specifically, this test checks that the column type becomes varchar(30) and a unique constraint on the 'bar' column gets added. These changes help implement a rule that every entry in the 'bar' column must be unique and of a certain length. This ensures the consistency and integrity of our data.

@codecov

codecov Bot commented Jul 28, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage has no change and project coverage change: +0.16% 🎉

Comparison is base (e5de2fc) 99.83% compared to head (ed6a37b) 100.00%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master      #304      +/-   ##
=============================================
+ Coverage     99.83%   100.00%   +0.16%     
  Complexity      201       201              
=============================================
  Files            13        13              
  Lines           599       599              
=============================================
+ Hits            598       599       +1     
+ Misses            1         0       -1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Tigrov Tigrov marked this pull request as ready for review July 28, 2023 06:22
@vjik vjik added type:test Test status:code review The pull request needs review. labels Jul 28, 2023
@vjik vjik requested a review from a team July 28, 2023 08:01
@vjik vjik merged commit 91bc69f into yiisoft:master Jul 28, 2023
@vjik

vjik commented Jul 28, 2023

Copy link
Copy Markdown
Member

@Tigrov Thanks 👍

@Tigrov Tigrov deleted the add_test_for_column_interface branch July 28, 2023 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review. type:test Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants