Merge pull request #1 from cravindra/scoped-radio-groups

Scoped radio groups

Author: cravindra

.gitignore

@@ -8,3 +8,4 @@ test/e2e/reports
 stats.json
 typings/
 typings.json
+.idea/

src/fields/core/fieldRadios.vue

@@ -23,7 +23,7 @@ export default {
 			}
 		},
 		id() {
-			return this.schema.model;
+			return this.getFieldID(this.schema, true);
 		}
 	},
 

test/unit/specs/fields/fieldRadios.spec.js

@@ -123,6 +123,24 @@ describe("FieldRadios.vue", () => {
 				expect(labelList.at(6).classes()).to.not.include("is-checked");
 			});
 		});
+
+		describe("test 'name' attribute for the radio group", () => {
+			it("should render the same 'name' attribute for all input[type=radio] within an instance", () => {
+				const nameAttribute = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute}"]`));
+			});
+			it("should render different 'name' attribute for all input[type=radio] across different instances", () => {
+				const nameAttribute = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute}"]`));
+
+				createField2({ schema, model, disabled: false });
+
+				const nameAttribute2 = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute2}"]`));
+
+				expect(nameAttribute).not.to.equal(nameAttribute2);
+			});
+		});
 	});
 
 	describe("check static values with { value, name } objects (default key name)", () => {
@@ -221,6 +239,24 @@ describe("FieldRadios.vue", () => {
 				expect(labelList.at(6).classes()).to.not.include("is-checked");
 			});
 		});
+
+		describe("test 'name' attribute for the radio group", () => {
+			it("should render the same 'name' attribute for all input[type=radio] within an instance", () => {
+				const nameAttribute = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute}"]`));
+			});
+			it("should render different 'name' attribute for all input[type=radio] across different instances", () => {
+				const nameAttribute = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute}"]`));
+
+				createField2({ schema, model, disabled: false });
+
+				const nameAttribute2 = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute2}"]`));
+
+				expect(nameAttribute).not.to.equal(nameAttribute2);
+			});
+		});
 	});
 
 	describe("check static values with { id, label } objects (custom key name with `radiosOptions`)", () => {
@@ -323,5 +359,23 @@ describe("FieldRadios.vue", () => {
 				expect(labelList.at(6).classes()).to.not.include("is-checked");
 			});
 		});
+
+		describe("test 'name' attribute for the radio group", () => {
+			it("should render the same 'name' attribute for all input[type=radio] within an instance", () => {
+				const nameAttribute = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute}"]`));
+			});
+			it("should render different 'name' attribute for all input[type=radio] across different instances", () => {
+				const nameAttribute = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute}"]`));
+
+				createField2({ schema, model, disabled: false });
+
+				const nameAttribute2 = radios.at(0).attributes().name;
+				expect(radios.is(`input[name="${nameAttribute2}"]`));
+
+				expect(nameAttribute).not.to.equal(nameAttribute2);
+			});
+		});
 	});
 });