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.
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
chore(warehouse): integration test changes v3 #2426
Conversation
… staging files, load files, table uploads for warehouse integration tests.
…re.warehouseIntegrationtTestV3
Codecov Report
@@ Coverage Diff @@
## master #2426 +/- ##
==========================================
+ Coverage 36.34% 36.56% +0.22%
==========================================
Files 200 200
Lines 39843 39843
==========================================
+ Hits 14479 14567 +88
+ Misses 24454 24360 -94
- Partials 910 916 +6
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
…re.warehouseIntegrationtTestV3
…re.warehouseIntegrationtTestV3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR includes major refactoring on the testcases, so I'll approve without much going in - depth.
"fmt" | ||
"log" | ||
"os" | ||
"testing" | ||
|
||
backendconfig "github.com/rudderlabs/rudder-server/config/backend-config" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary space ?
Description
This contains series of changes related to Warehouse Integration Test:
wh_**
) tables instead ofgw_**
orbrt_**
tables.staging files
,load files
,table_uploads
, andwarehouse
.gw
andbrt
dependency.Notion Ticket
https://www.notion.so/rudderstacks/Integration-tests-v3-c9dcc33c577b4cb5806155429ff15378
Security