Fix false-positive in no-ambiguity-target, fixes #133

rules/no-ambiguity-target/examples/correct-example-issue-133.js

@@ -0,0 +1,16 @@
+// Examples were found in production code-base with exception on 0.10.3
+// https://github.com/igorkamyshev/eslint-plugin-effector/issues/133
+
+import { createStore, createEvent, sample } from "effector";
+
+const obj = {
+  fn: () => {
+    const $store = createStore(0);
+    const event = createEvent();
+    // warning  Method `sample` returns `target` and assigns the result to a variable. Consider removing one of them  effector/no-ambiguity-target
+    sample({
+      source: event,
+      target: $store,
+    });
+  },
+};

rules/no-ambiguity-target/no-ambiguity-target.js

@@ -53,7 +53,8 @@ module.exports = {
         );
         const resultPartOfChain = traverseParentByType(
           node,
-          "ObjectExpression"
+          "ObjectExpression",
+          { stopOnTypes: ["BlockStatement"] }
         );
 
         if (resultAssignedInVariable || resultPartOfChain) {

rules/no-ambiguity-target/no-ambiguity-target.test.js

@@ -0,0 +1,23 @@
+const { RuleTester } = require("eslint");
+const { join } = require("path");
+
+const { readExample } = require("../../utils/read-example");
+const rule = require("./no-ambiguity-target");
+
+const ruleTester = new RuleTester({
+  parserOptions: {
+    ecmaVersion: 2020,
+    sourceType: "module",
+  },
+});
+
+const readExampleForTheRule = (name) => ({
+  code: readExample(__dirname, name),
+  filename: join(__dirname, "examples", name),
+});
+
+ruleTester.run("effector/no-ambiguity-target.js.test", rule, {
+  valid: ["correct-example-issue-133.js"].map(readExampleForTheRule),
+
+  invalid: [],
+});