Skip to content

Commit

Permalink
change eslint settings and an integ test
Browse files Browse the repository at this point in the history
  • Loading branch information
go-to-k committed Feb 8, 2024
1 parent 91d0a2f commit 55fbb20
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
1 change: 1 addition & 0 deletions packages/@aws-cdk/aws-scheduler-targets-alpha/.eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,6 @@ const baseConfig = require('@aws-cdk/cdk-build-tools/config/eslintrc');
baseConfig.parserOptions.project = __dirname + '/tsconfig.json';

baseConfig.rules['import/no-extraneous-dependencies'] = ['error', { devDependencies: true, peerDependencies: true }];
baseConfig.rules['@aws-cdk/invalid-cfn-imports'] = 'off';

module.exports = baseConfig;
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,15 @@ import { ExpectedResult, IntegTest } from '@aws-cdk/integ-tests-alpha';
import * as cdk from 'aws-cdk-lib';
import { Grant, IGrantable, ManagedPolicy, Role, ServicePrincipal } from 'aws-cdk-lib/aws-iam';
import { Bucket } from 'aws-cdk-lib/aws-s3';
import { CfnPipeline } from 'aws-cdk-lib/aws-sagemaker';
import { IPipeline } from 'aws-cdk-lib/aws-sagemaker';
import * as sagemaker from 'aws-cdk-lib/aws-sagemaker';
import { Construct } from 'constructs';
import { SageMakerPipelineParameter, SageMakerStartPipelineExecution } from '../lib';

interface FakePipelineProps {
readonly pipelineName: string;
}

class FakePipeline extends cdk.Resource implements IPipeline {
class FakePipeline extends cdk.Resource implements sagemaker.IPipeline {
public readonly pipelineArn;

public readonly pipelineName;
Expand Down Expand Up @@ -82,7 +81,7 @@ class FakePipeline extends cdk.Resource implements IPipeline {
});
pipelineRole.addManagedPolicy(ManagedPolicy.fromAwsManagedPolicyName('AmazonSageMakerFullAccess'));

const pipeline = new CfnPipeline(this, 'Resource', {
const pipeline = new sagemaker.CfnPipeline(this, 'Resource', {
pipelineName: this.pipelineName,
pipelineDefinition: pipelineDefinition,
roleArn: pipelineRole.roleArn,
Expand Down

0 comments on commit 55fbb20

Please sign in to comment.