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

chore(release): pull release/v1.47.0 into main #2779

Merged
merged 20 commits into from
Oct 31, 2023
Merged

chore(release): pull release/v1.47.0 into main #2779

merged 20 commits into from
Oct 31, 2023

Conversation

devops-github-rudderstack
Copy link
Contributor

👑 An automated PR

sanpj2292 and others added 18 commits October 18, 2023 14:36
@sanpj2292
chore: refactor migrate test cases to new format (#2630)
dca6ee0
@mihir-4116
chore(release): pull main into develop post release v1.46.0 (#2743)
de2be59
@yashasvibajpai @ujjwal-ab
refactor: destination tests to component test case suite (#2698)
5f094bc 
* refactor: algolia, appcues, attentive to component tc

* test: fix processor tc for branch,candu,customerio

* test: add proc cases for autopilot,azure,criteo,facebookpixel

* test: add proc cases for blueshift,cm,confluent,courier,fboc

* test: add router cases for all destinations

* fix: timestamp testcase in fb pixel

* chore: cleanup old test and data files

* refactor: revert formattings

* refactor: revert formattingsx2

---------

Co-authored-by: Ujjwal Abhishek <[email protected]>
@koladilip
feat: add support filtering component tests by feature and index (#2748)
146f1c6 
* feat: add support filtering component tests by feature and index

* feat: add support filtering component tests by feature and index
@sanpj2292
fix: bugsnag error for salesforce (#2753)
a2ccdad
@chandumlg
chore(release): pull main into develop post release v1.46.1 (#2751)
81066ed
@mihir-4116
chore(release): pull main into develop post release v1.46.3 (#2759)
9ead1b6
@mihir-4116
chore(release): pull main into develop post release v1.46.4 (#2762)
656ba27
@mihir-4116
chore(release): pull main into develop post release v1.46.5 (#2767)
4079ec6
@anantjain45823
fix: salesforce: handle ECONNABORTED error (#2732)
6b23a9b 
* fix: salesforce: handle ECONNABORTED error

* chore: added status codes to universal error code handler

* chore: added error codes to salesforce error code handler

* fix: using correct error response

* comment addressed+1
@ItsSudip
refactor: migrate ga, ga360, gcs_datalake, google adwords destination…
52c432c 
…s to component tests (#2675)
@sanpj2292
fix: oom kill while stringifying large response json (#2754)
c8baf5b
@ujjwal-ab
feat: onboard destination ortto (#2730)
9be5740 
* feat: onboard destination ortto

* feat: add track call support

* feat: add router batching support

* fix: refactor code

* fix: add instance region support

* fix: fix merge by values

* fix: add func for birthday obj

* fix: fix header

* fix: fix header

* fix: remove logger

* fix: property mapping

* fix: property mapping

* fix: refactor code

* fix: add router tests

* fix: update tests

* fix: refactor code

* fix: update tags mapping

* fix: update tags mapping

* chore: refactor code and add tests

* fix: code smell

* fix: refactor birthday validation
@Sanjay-Veernala @utsabc @krishna2020
chore(INT-562): update component test suite (#2744)
3387cb0 
* component tests(chore): update component test data

* componentTestSuite(chore): remove unwanted files

* chore: clean-up un-used tests

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: clean-up vero tests

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: snapchat tests, remove unnecessary tests for snapchat
- remove ts-ignore for attentive_tag processor tests

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: data-delivery component tests for salesforce

* fix: remove salesforce from original proxy tests & correct the migrated transformation tests

Signed-off-by: Sai Sankeerth <[email protected]>

* migrate marketo-proxy tests to component tests

* chore: migrating intercom, braze, marketo_static_list, snapchat_custom_audience, fb, fbpixel, clevertap, criteo_audience

* fix: move tiktok_ads data-delivery tests to component test suite

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: mock clone

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: decreasing the code-coverage to 75%

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: moving am to component test suite to cover generic handler tests

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: remove proxy.test

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: clean-up braze mocks, un-comment twitter_ads processor test-cases

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: add router component tests for tiktok offline events

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: salesforce router test-case format for router component tests

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: tiktok_ads router test-case format in component tests

Signed-off-by: Sai Sankeerth <[email protected]>

* fix: vero, splitio router test-case format in component tests

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: move back threshold for coverage to 80

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: move threshold for coverage to 75

Signed-off-by: Sai Sankeerth <[email protected]>

* chore: updated test case descriptions

---------

Signed-off-by: Sai Sankeerth <[email protected]>
Co-authored-by: Sai Sankeerth <[email protected]>
Co-authored-by: Utsab Chowdhury <[email protected]>
Co-authored-by: krishnachaitanya <[email protected]>
@Gauravudia
feat: add custom utm parameters to mixpanel (#2771)
9c4fcd3 
* feat: add custom utm parameters to mixpanel

* test: add testcases

* refactor: add optional chaining

* refactor: add isObject check
@shrouti1507 @ujjwal-ab
feat: onboard facebook custom audience to transformer proxy (#2769)
14c5e40 
* feat: error codes added in fbUtils

* adding single test case

* adding test cases in new format

* adding test cases in new format

* fix: fixing the test cases

* fix: cleaning up files

* fix: cleaning up files

* fix: cleaning up files

* fix: review comments addressed

* fix: review comments addressed

* Empty commit

* fix: deleting wrongly spelled file

* fix: add network handler for fb_custom_audience with correct spelling

---------

Co-authored-by: Ujjwal Abhishek <[email protected]>
Co-authored-by: Sai Sankeerth <[email protected]>
@yashasvibajpai
chore: add yashasvibajpai and sanpj2292 to release workflow actors (#…
748b622 
…2778)
@web-flow
chore(release): 1.47.0
a210cbd
@yashasvibajpai yashasvibajpai self-assigned this Oct 30, 2023
@codecov
Copy link

codecov bot commented Oct 30, 2023 •
edited
Loading

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (2ba4864) 87.36% compared to head (f641a1e) 87.13%.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2779      +/-   ##
==========================================
- Coverage   87.36%   87.13%   -0.23%     
==========================================
  Files         646      766     +120     
  Lines       28588    28443     -145     
  Branches     6786     6655     -131     
==========================================
- Hits        24975    24785     -190     
- Misses       3288     3321      +33     
- Partials      325      337      +12
Files Coverage Δ
src/adapters/utils/networkUtils.js 73.58% <ø> (ø)
src/cdk/v2/destinations/eloqua/config.js 100.00% <ø> (ø)
src/cdk/v2/destinations/ortto/config.js 100.00% <100.00%> (ø)
src/routes/bulkUpload.ts 100.00% <100.00%> (ø)
src/routes/testEvents.ts 100.00% <100.00%> (ø)
src/util/prometheus.js 7.07% <ø> (ø)
src/v0/destinations/braze/networkHandler.js 100.00% <100.00%> (ø)
src/v0/destinations/clevertap/transform.js 98.30% <ø> (ø)
.../destinations/fb_custom_audience/networkHandler.js 100.00% <100.00%> (ø)
...tions/google_adwords_offline_conversions/helper.js 100.00% <100.00%> (ø)
... and 43 more

... and 128 files with indirect coverage changes

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

sanpj2292
sanpj2292 previously approved these changes Oct 31, 2023
View reviewed changes
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@sanpj2292 sanpj2292 self-requested a review October 31, 2023 07:24
@yashasvibajpai yashasvibajpai merged commit c147906 into main Oct 31, 2023
16 of 18 checks passed
@devops-github-rudderstack devops-github-rudderstack deleted the release/v1.47.0 branch October 31, 2023 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anantjain45823 anantjain45823 anantjain45823 approved these changes

@shrouti1507 shrouti1507 shrouti1507 approved these changes

@sanpj2292 sanpj2292 sanpj2292 approved these changes

@Jayachand Jayachand Jayachand approved these changes

@krishna2020 krishna2020 krishna2020 approved these changes

@sivashanmukh sivashanmukh Awaiting requested review from sivashanmukh sivashanmukh is a code owner

@sandeepdsvs sandeepdsvs Awaiting requested review from sandeepdsvs sandeepdsvs is a code owner automatically assigned from rudderlabs/integrations

Assignees

@yashasvibajpai yashasvibajpai

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.