Skip to content

Commit a4fc3ae

Browse files
authored
fix: INTMDB-1155 Atlas Basic L3 ip access list made mandatory (#147)
1 parent 3c26424 commit a4fc3ae

File tree

4 files changed

+21
-12
lines changed

4 files changed

+21
-12
lines changed

API.md

Lines changed: 10 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/l3-resources/atlas-basic/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,6 @@ export class AtlasBasic extends Construct {
114114
{
115115
profile: props.profile,
116116
projectId: this.mProject.attrId,
117-
accessList: props.ipAccessListProps?.accessList,
118117
...props.ipAccessListProps,
119118
}
120119
);

src/l3-resources/common/props.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ export interface AtlasBasicProps {
4040
* @type {IpAccessListProps}
4141
* @memberof AtlasBasicProps
4242
*/
43-
readonly ipAccessListProps?: IpAccessListProps;
43+
readonly ipAccessListProps: IpAccessListProps;
4444
}
4545

4646
export interface AtlasServerlessBasicProps {

test/l3-resources/atlas-basic/index.test.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@ import * as l3 from "../../../src";
1919
const RESOURCE_NAME_PROJECT = "MongoDB::Atlas::Project";
2020
const RESOURCE_NAME_CLUSTER = "MongoDB::Atlas::Cluster";
2121
const RESOURCE_NAME_DB_USER = "MongoDB::Atlas::DatabaseUser";
22+
const RESOURCE_NAME_PROJECT_IP_ACCESS_LIST =
23+
"MongoDB::Atlas::ProjectIpAccessList";
2224
const PROJECT_ID = "testProjectId";
2325
const ORG_ID = "testProjectId";
2426
const PROJECT_NAME = "test";
@@ -29,6 +31,7 @@ const DATABASE_USER_NAME = "atlas-user";
2931
const ADMIN_DB = "admin";
3032
const ROLE_NAME = "atlasAdmin";
3133
const PWD = "test";
34+
const IP_ACCESS = "0000";
3235

3336
test("AtlasBasis construct should contain default properties", () => {
3437
const mockApp = new App();
@@ -61,6 +64,9 @@ test("AtlasBasis construct should contain default properties", () => {
6164
databaseName: DATABASE_NAME,
6265
password: PWD,
6366
},
67+
ipAccessListProps: {
68+
accessList: [{ ipAddress: IP_ACCESS, comment: "My first IP address" }],
69+
},
6470
});
6571

6672
const template = Template.fromStack(stack);
@@ -101,4 +107,8 @@ test("AtlasBasis construct should contain default properties", () => {
101107
},
102108
],
103109
});
110+
111+
template.hasResourceProperties(RESOURCE_NAME_PROJECT_IP_ACCESS_LIST, {
112+
AccessList: [{ IPAddress: IP_ACCESS, Comment: "My first IP address" }],
113+
});
104114
});

0 commit comments

Comments
 (0)