[ https://issues.apache.org/jira/browse/NIFI-3931?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16076909#comment-16076909 ]
ASF GitHub Bot commented on NIFI-3931: -------------------------------------- Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1968#discussion_r125958747 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/SFTPIT.java --- @@ -0,0 1,51 @@ /* * Licensed to the Apache Software Foundation (ASF) under one or more * contributor license agreements. See the NOTICE file distributed with * this work for additional information regarding copyright ownership. * The ASF licenses this file to You under the Apache License, Version 2.0 * (the "License"); you may not use this file except in compliance with * the License. You may obtain a copy of the License at * * http://www.apache.org/licenses/LICENSE-2.0 * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. * See the License for the specific language governing permissions and * limitations under the License. */ package org.apache.nifi.processors.standard; import org.apache.nifi.processors.standard.util.SFTPTransfer; import org.apache.nifi.util.TestRunner; import org.apache.nifi.util.TestRunners; import org.junit.Test; public class SFTPIT { --- End diff -- The tests executed during the builds are controlled by the surefire plugin definition in the top level pom file: ````xml <includes> <include>**/*Test.class</include> <include>**/Test*.class</include> <include>**/*Spec.class</include> </includes> ```` It should not be necessary to add ``@Ignore`` in this class. I believe the convention is to use IT (and to not use Test) in the class name when implementing integration tests. > putSftp process port property should support for expression language > -------------------------------------------------------------------- > > Key: NIFI-3931 > URL: https://issues.apache.org/jira/browse/NIFI-3931 > Project: Apache NiFi > Issue Type: Improvement > Affects Versions: 1.2.0 > Reporter: Cheng Chin Tat > Assignee: Pierre Villard > Priority: Minor > Labels: easyfix > Attachments: TestFTP.xml > > > PutSftp Processor port property should support for expression language so > that dynamic port number can be pass to the processor during run time. > Rather than preset the port on design time. > This changes involve changing the PropertyDescriptor SFTP_PORT validator > StandardValidators.NON_NEGATIVE_INTEGER_VALIDATOR to > StandardValidators.NON_EMPTY_VALIDATOR and other codes. -- This message was sent by Atlassian JIRA (v6.4.14#64029)