123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394 |
- import {
- render,
- screen,
- userEvent,
- waitFor,
- within,
- } from 'sentry-test/reactTestingLibrary';
- import {SERVER_SIDE_SAMPLING_DOC_LINK} from 'sentry/views/settings/project/server-side-sampling/utils';
- import {samplingBreakdownTitle} from './samplingBreakdown.spec';
- import {
- getMockData,
- mockedProjects,
- mockedSamplingDistribution,
- mockedSamplingSdkVersions,
- specificRule,
- TestComponent,
- uniformRule,
- } from './testUtils';
- describe('Server-Side Sampling', function () {
- let distributionMock: ReturnType<typeof MockApiClient.addMockResponse> | undefined =
- undefined;
- let sdkVersionsMock: ReturnType<typeof MockApiClient.addMockResponse> | undefined =
- undefined;
- beforeEach(function () {
- distributionMock = MockApiClient.addMockResponse({
- url: '/projects/org-slug/project-slug/dynamic-sampling/distribution/',
- method: 'GET',
- body: mockedSamplingDistribution,
- });
- sdkVersionsMock = MockApiClient.addMockResponse({
- url: '/organizations/org-slug/dynamic-sampling/sdk-versions/',
- method: 'GET',
- body: mockedSamplingSdkVersions,
- });
- MockApiClient.addMockResponse({
- url: '/organizations/org-slug/projects/',
- method: 'GET',
- body: mockedSamplingDistribution.project_breakdown!.map(p =>
- TestStubs.Project({id: p.project_id, slug: p.project})
- ),
- });
- MockApiClient.addMockResponse({
- url: '/organizations/org-slug/stats_v2/',
- method: 'GET',
- body: TestStubs.Outcomes(),
- });
- });
- afterEach(() => {
- MockApiClient.clearMockResponses();
- });
- it('renders onboarding promo', async function () {
- const {router, organization, project} = getMockData();
- const {container} = render(
- <TestComponent router={router} organization={organization} project={project} />
- );
- expect(
- screen.getByRole('heading', {name: /Server-Side Sampling/})
- ).toBeInTheDocument();
- expect(screen.getByText(/enhance the performance monitoring/i)).toBeInTheDocument();
- // Assert that project breakdown is there
- expect(await screen.findByText(samplingBreakdownTitle)).toBeInTheDocument();
- expect(
- screen.getByRole('heading', {name: 'Sample for relevancy'})
- ).toBeInTheDocument();
- expect(
- screen.getByText(
- 'Create rules to sample transactions under specific conditions, keeping what you need and dropping what you don’t.'
- )
- ).toBeInTheDocument();
- expect(screen.getByRole('button', {name: 'Read Docs'})).toHaveAttribute(
- 'href',
- SERVER_SIDE_SAMPLING_DOC_LINK
- );
- expect(screen.getByRole('button', {name: 'Start Setup'})).toBeInTheDocument();
- expect(container).toSnapshot();
- });
- it('renders rules panel', async function () {
- const {router, organization, project} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [{...uniformRule, sampleRate: 1}],
- },
- }),
- ],
- });
- const {container} = render(
- <TestComponent router={router} organization={organization} project={project} />
- );
- // Assert that project breakdown is there
- expect(await screen.findByText(samplingBreakdownTitle)).toBeInTheDocument();
- // Rule Panel Header
- expect(screen.getByText('Operator')).toBeInTheDocument();
- expect(screen.getByText('Condition')).toBeInTheDocument();
- expect(screen.getByText('Rate')).toBeInTheDocument();
- expect(screen.getByText('Active')).toBeInTheDocument();
- // Rule Panel Content
- expect(screen.getAllByTestId('sampling-rule').length).toBe(1);
- expect(screen.queryByLabelText('Drag Rule')).not.toBeInTheDocument();
- expect(screen.getByTestId('sampling-rule')).toHaveTextContent('If');
- expect(screen.getByTestId('sampling-rule')).toHaveTextContent('All');
- expect(screen.getByTestId('sampling-rule')).toHaveTextContent('100%');
- expect(screen.getByLabelText('Activate Rule')).toBeInTheDocument();
- expect(screen.getByLabelText('Actions')).toBeInTheDocument();
- // Rule Panel Footer
- expect(screen.getByText('Add Rule')).toBeInTheDocument();
- expect(screen.getByRole('button', {name: 'Read Docs'})).toHaveAttribute(
- 'href',
- SERVER_SIDE_SAMPLING_DOC_LINK
- );
- expect(container).toSnapshot();
- });
- it('does not let you delete the base rule', async function () {
- const {router, organization, project} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [
- {
- sampleRate: 0.2,
- type: 'trace',
- active: false,
- condition: {
- op: 'and',
- inner: [
- {
- op: 'glob',
- name: 'trace.release',
- value: ['1.2.3'],
- },
- ],
- },
- id: 2,
- },
- {
- sampleRate: 0.2,
- type: 'trace',
- active: false,
- condition: {
- op: 'and',
- inner: [],
- },
- id: 1,
- },
- ],
- next_id: 3,
- },
- }),
- ],
- });
- render(
- <TestComponent router={router} organization={organization} project={project} />
- );
- // Assert that project breakdown is there (avoids 'act' warnings)
- expect(await screen.findByText(samplingBreakdownTitle)).toBeInTheDocument();
- const deleteButtons = screen.getAllByLabelText('Delete');
- expect(deleteButtons[0]).not.toHaveAttribute('disabled'); // eslint-disable-line jest-dom/prefer-enabled-disabled
- expect(deleteButtons[1]).toHaveAttribute('disabled'); // eslint-disable-line jest-dom/prefer-enabled-disabled
- });
- it('display "update sdk versions" alert and open "recommended next step" modal', async function () {
- const {organization, projects, router} = getMockData({
- projects: mockedProjects,
- });
- render(
- <TestComponent
- organization={organization}
- project={projects[2]}
- router={router}
- withModal
- />
- );
- expect(distributionMock).toHaveBeenCalled();
- await waitFor(() => {
- expect(sdkVersionsMock).toHaveBeenCalled();
- });
- const recommendedSdkUpgradesAlert = await screen.findByTestId(
- 'recommended-sdk-upgrades-alert'
- );
- expect(
- within(recommendedSdkUpgradesAlert).getByText(
- 'To activate server-side sampling rules, it’s a requirement to update the following project SDK(s):'
- )
- ).toBeInTheDocument();
- expect(
- within(recommendedSdkUpgradesAlert).getByRole('link', {
- name: mockedProjects[1].slug,
- })
- ).toHaveAttribute(
- 'href',
- `/organizations/org-slug/projects/sentry/?project=${mockedProjects[1].id}`
- );
- // Open Modal
- userEvent.click(
- within(recommendedSdkUpgradesAlert).getByRole('button', {
- name: 'Learn More',
- })
- );
- expect(await screen.findByRole('heading', {name: 'Next steps'})).toBeInTheDocument();
- });
- it('Open specific conditions modal', async function () {
- const {router, project, organization} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [
- {
- sampleRate: 1,
- type: 'trace',
- active: false,
- condition: {
- op: 'and',
- inner: [],
- },
- id: 1,
- },
- ],
- },
- }),
- ],
- });
- render(
- <TestComponent
- organization={organization}
- project={project}
- router={router}
- withModal
- />
- );
- // Open Modal
- userEvent.click(screen.getByLabelText('Add Rule'));
- expect(await screen.findByRole('heading', {name: 'Add Rule'})).toBeInTheDocument();
- });
- it('does not let user add without permissions', async function () {
- const {organization, router, project} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [uniformRule],
- },
- }),
- ],
- access: [],
- });
- render(
- <TestComponent organization={organization} project={project} router={router} />
- );
- expect(screen.getByRole('button', {name: 'Add Rule'})).toBeDisabled();
- userEvent.hover(screen.getByText('Add Rule'));
- expect(
- await screen.findByText("You don't have permission to add a rule")
- ).toBeInTheDocument();
- expect(distributionMock).not.toHaveBeenCalled();
- expect(sdkVersionsMock).not.toHaveBeenCalled();
- });
- it('does not let the user activate a rule if sdk updates exists', async function () {
- const {organization, router, project} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [uniformRule],
- },
- }),
- ],
- });
- render(
- <TestComponent organization={organization} project={project} router={router} />
- );
- await screen.findByTestId('recommended-sdk-upgrades-alert');
- expect(screen.getByRole('checkbox', {name: 'Activate Rule'})).toBeDisabled();
- userEvent.hover(screen.getByLabelText('Activate Rule'));
- expect(
- await screen.findByText(
- 'To enable the rule, the recommended sdk version have to be updated'
- )
- ).toBeInTheDocument();
- });
- it('open uniform rate modal when editing a uniform rule', async function () {
- const {organization, router, project} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [uniformRule],
- },
- }),
- ],
- });
- render(
- <TestComponent
- organization={organization}
- project={project}
- router={router}
- withModal
- />
- );
- userEvent.click(screen.getByLabelText('Actions'));
- // Open Modal
- userEvent.click(screen.getByLabelText('Edit'));
- expect(
- await screen.findByRole('heading', {
- name: 'Set a global sample rate',
- })
- ).toBeInTheDocument();
- });
- it('does not let user reorder uniform rule', async function () {
- const {organization, router, project} = getMockData({
- projects: [
- TestStubs.Project({
- dynamicSampling: {
- rules: [specificRule, uniformRule],
- },
- }),
- ],
- });
- render(
- <TestComponent
- organization={organization}
- project={project}
- router={router}
- withModal
- />
- );
- const samplingUniformRule = screen.getAllByTestId('sampling-rule')[1];
- expect(
- within(samplingUniformRule).getByRole('button', {name: 'Drag Rule'})
- ).toHaveAttribute('aria-disabled', 'true');
- userEvent.hover(within(samplingUniformRule).getByLabelText('Drag Rule'));
- expect(
- await screen.findByText('Uniform rules cannot be reordered')
- ).toBeInTheDocument();
- });
- });
|