MAGETWO-53714: BNCODE is not send in PayPal solutions properly for CE and EE

Author: oshmyheliuk

app/code/Magento/Paypal/Model/AbstractConfig.php

@@ -54,7 +54,7 @@ abstract class AbstractConfig implements ConfigInterface
     /**
      * @var ProductMetadataInterface
      */
-    private $productMetadata;
+    protected $productMetadata;
 
     /**
      * @var string

app/code/Magento/Paypal/Model/Payflowpro.php

@@ -623,7 +623,7 @@ public function buildBasicRequest()
         $request->setPartner($this->getConfigData('partner'));
         $request->setPwd($this->getConfigData('pwd'));
         $request->setVerbosity($this->getConfigData('verbosity'));
-        $request->setData('BNCODE', $config->getBuildNotationCode());
+        $request->setData('BUTTONSOURCE', $config->getBuildNotationCode());
         $request->setTender(self::TENDER_CC);
 
         return $request;

app/code/Magento/Paypal/Test/Unit/Model/AbstractConfigTest.php

@@ -8,6 +8,8 @@
 use Magento\Framework\App\Config\ScopeConfigInterface;
 use Magento\Store\Model\ScopeInterface as ModelScopeInterface;
 use Magento\Payment\Model\MethodInterface;
+use Magento\Framework\App\ProductMetadataInterface;
+use Magento\Framework\TestFramework\Unit\Helper\ObjectManager as ObjectManagerHelper;
 
 /**
  * Class AbstractConfigTest
@@ -290,10 +292,18 @@ public function testIsMethodActive()
 
     public function testGetBuildNotationCode()
     {
-        $this->scopeConfigMock->expects($this->once())
-            ->method('getValue')
-            ->with('paypal/bncode');
+        $productMetadata = $this->getMock(ProductMetadataInterface::class, [], [], '', false);
+        $productMetadata->expects($this->once())
+            ->method('getEdition')
+            ->will($this->returnValue('SomeEdition'));
+
+        $objectManagerHelper = new ObjectManagerHelper($this);
+        $objectManagerHelper->setBackwardCompatibleProperty(
+            $this->config,
+            'productMetadata',
+            $productMetadata
+        );
 
-        $this->config->getBuildNotationCode();
+        $this->assertEquals('Magento_Cart_SomeEdition', $this->config->getBuildNotationCode());
     }
 }

app/code/Magento/Paypal/Test/Unit/Model/ConfigTest.php

@@ -99,17 +99,6 @@ public function testGetCountryMethods()
         $this->assertNotContains(Config::METHOD_WPP_BML, $this->model->getCountryMethods('DE'));
     }
 
-    public function testGetBuildNotationCode()
-    {
-        $this->model->setMethod('payflow_direct');
-        $this->model->setStoreId(123);
-        $this->scopeConfig->expects($this->once())
-            ->method('getValue')
-            ->with('paypal/bncode', ScopeInterface::SCOPE_STORE, 123)
-            ->will($this->returnValue('some BN code'));
-        $this->assertEquals('some BN code', $this->model->getBuildNotationCode());
-    }
-
     public function testIsMethodActive()
     {
         $this->assertFalse($this->model->isMethodActive('payflow_direct'));

app/code/Magento/Paypal/Test/Unit/Model/PayflowlinkTest.php

@@ -146,7 +146,7 @@ public function testInitialize()
                         'partner' => null,
                         'pwd' => null,
                         'verbosity' => null,
-                        'BNCODE' => 'build notation code',
+                        'BUTTONSOURCE' => 'build notation code',
                         'tender' => 'C',
                     ],
                     $this->returnSelf()